First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 168386
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: AMD64 Project <amd64@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Michael A. Smith <michael@smith-li.com>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
emul-linux-x86-baselibs-10.1-r1.ebuild emul-linux-x86-baselibs-10.1.ebuild text/plain Michael A. Smith 2007-02-25 22:14 0000 2.91 KB Details
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 168386 depends on: Show dependency tree
Bug 168386 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2007-02-25 22:13 0000
There's a bit of odd code in app-emulation/emul-linux-x86-10.1.ebuild. Here's
an alternative solution that IMO is a bit cleaner.

Reproducible: Always

Steps to Reproduce:

------- Comment #1 From Michael A. Smith 2007-02-25 22:14:41 0000 -------
Created an attachment (id=111238) [edit]
emul-linux-x86-baselibs-10.1.ebuild

---
app-emulation/emul-linux-x86-baselibs/emul-linux-x86-baselibs-10.1.ebuild.orig 
      2007-02-19 17:05:33.000000000 -0500
+++ app-emulation/emul-linux-x86-baselibs/emul-linux-x86-baselibs-10.1.ebuild  
  2007-02-23 13:45:23.000000000 -0500
@@ -64,13 +64,16 @@
        unpack ${A}
        cd ${S}

+       # rm these first so find has less to do
+       rm -r "${S}/etc/env.d/binutils/" \
+             "${S}/usr/lib32/binutils/" \
+             "${S}/usr/lib32/engines/"  \
+             "${S}/usr/lib32/openldap/" \
+             "${S}/usr/lib32/python2.4/"
+
+       # Not POSIX compliant, but it should work on amd64
        WHITELIST="(${S}/lib32/security/pam_filter/upperLOWER|${S}/etc/env.d)"
-       # the following line is broken up because we have files with spaces in
the filename
-       find ${S} ! -type d ! -name '*.so*' | egrep -v "${WHITELIST}" | xargs
-d '
-' rm -f
-       rm -rf ${S}/etc/env.d/binutils/ ${S}/usr/lib32/binutils/
-       rm -rf ${S}/usr/lib32/engines/ ${S}/usr/lib32/openldap/
-       rm -rf ${S}/usr/lib32/python2.4/
+       find ${S} ! -type d ! -name '*.so*' -print0 | egrep --null -v
"${WHITELIST}" | xargs -0 rm

        ln -s ../share/terminfo ${S}/usr/lib32/terminfo
 }

------- Comment #2 From Simon Stelling (RETIRED) 2007-02-26 10:02:47 0000 -------
Actually 10.1-r1 was already put in the tree which uses the eclass so looks
much cleaner, but i merged the rm calls together. Thanks.

First Last Prev Next    No search results available      Search page      Enter new bug