Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 168386 - Strange newline in app-emulation/emul-linux-x86-baselibs-10.1
Summary: Strange newline in app-emulation/emul-linux-x86-baselibs-10.1
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: AMD64 Linux
: High trivial (vote)
Assignee: AMD64 Project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-02-25 22:13 UTC by michael@smith-li.com
Modified: 2007-02-26 10:02 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
emul-linux-x86-baselibs-10.1.ebuild (emul-linux-x86-baselibs-10.1-r1.ebuild,2.91 KB, text/plain)
2007-02-25 22:14 UTC, michael@smith-li.com
Details

Note You need to log in before you can comment on or make changes to this bug.
Description michael@smith-li.com 2007-02-25 22:13:01 UTC
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 michael@smith-li.com 2007-02-25 22:14:41 UTC
Created attachment 111238 [details]
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 Simon Stelling (RETIRED) gentoo-dev 2007-02-26 10:02:47 UTC
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.