Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Simon Stelling
Mike Frysinger wrote: replace that with: xargs -d $'\n' aye, that looks way better. -- Kind Regards, Simon Stelling Gentoo/AMD64 developer -- gentoo-dev@gentoo.org mailing list

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Mike Frysinger
On Monday 19 February 2007, Piotr Jaroszyński wrote: > If you are using regex here, why don't you use it in find? because it isnt a POSIX option -mike pgpm4PU4zcDIt.pgp Description: PGP signature

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Piotr Jaroszyński
ALLOWED=${ALLOWED:-^${S}/etc/env.d} If you are using regex here, why don't you use it in find? find "${S}" ! -type d ! -name '*.so*' ! -regex ${ALLOWED} -print0 | xargs -0 /bin/rm -f Note that you will need to change ^${S}/etc/env.d to ^${S}/etc/env\.d.* as it needs to be a full match( btw. you

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Mike Frysinger
On Monday 19 February 2007, Simon Stelling wrote: > Mike Frysinger wrote: > > every use of find | xargs in there should be fixed to use find -print0 | > > xargs -0 > > The second one, yes, that's fixed now. The first one, no, cause egrep > wouldn't like it. The xargs -d' > ' ensures that \n instead

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Simon Stelling
Mike Frysinger wrote: every use of find | xargs in there should be fixed to use find -print0 | xargs -0 The second one, yes, that's fixed now. The first one, no, cause egrep wouldn't like it. The xargs -d' ' ensures that \n instead of a simple space is used as delimiter. If some package reall

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Petteri Räty
Simon Stelling wrote: > > ECLASS="emul-libs" > Not needed any more: [EMAIL PROTECTED] ~ $ grep ECLASS= /usr/portage/eclass/* /usr/portage/eclass/ccc.eclass:#DEBUG_CCC_ECLASS=1 /usr/portage/eclass/xemacs-packages.eclass:ECLASS="xemacs-packages" /usr/portage/eclass/x-modular.eclass:FONT_ECLASS=""

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Simon Stelling
Fabian Groffen wrote: I'll use cp -pPr. Actually -dpPR, which is what -a is an alias for. Yes, but -d is a GNU option, and BSD people are after the POSIX only options, hence the -pPR. :) Missed that, Flameeyes just told me about it, so the -d will be dropped again ;) -- Kind Regards, Si

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Mike Frysinger
On Monday 19 February 2007, Simon Stelling wrote: > Thanks for any feedback, every use of find | xargs in there should be fixed to use find -print0 | xargs -0 -mike pgpGdDjhf8ch3.pgp Description: PGP signature

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Fabian Groffen
On 19-02-2007 18:12:42 +0100, Simon Stelling wrote: > Simon Stelling wrote: > >I'll use cp -pPr. > > Actually -dpPR, which is what -a is an alias for. Yes, but -d is a GNU option, and BSD people are after the POSIX only options, hence the -pPR. :) -- Fabian Groffen Gentoo on a different level

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Ciaran McCreesh
On Mon, 19 Feb 2007 18:03:36 +0100 Simon Stelling <[EMAIL PROTECTED]> wrote: | Timothy Redaelli wrote: | >># remove void directories | >>find "${S}" -depth -type d | xargs rmdir 2&>/dev/null | > | > Portage should remove blank dirs or am i wrong? | | Yes, but only in the unmerge phase it

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Simon Stelling
Simon Stelling wrote: I'll use cp -pPr. Actually -dpPR, which is what -a is an alias for. -- Kind Regards, Simon Stelling Gentoo/AMD64 developer -- gentoo-dev@gentoo.org mailing list

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Simon Stelling
Timothy Redaelli wrote: # remove void directories find "${S}" -depth -type d | xargs rmdir 2&>/dev/null Portage should remove blank dirs or am i wrong? Yes, but only in the unmerge phase it seems, so you it installs them, then checks whether they are empty and removes them ag

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Ciaran McCreesh
On Mon, 19 Feb 2007 17:05:01 +0100 Timothy Redaelli <[EMAIL PROTECTED]> wrote: | Mike Frysinger wrote: | > On Monday 19 February 2007, Fabian Groffen wrote: | >> On 19-02-2007 14:14:18 +0100, Timothy Redaelli wrote: | cp -a "${S}"/* "${D}"/ || die "copying files failed!" | >>> For future *BSD

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Timothy Redaelli
Mike Frysinger wrote: > On Monday 19 February 2007, Fabian Groffen wrote: >> On 19-02-2007 14:14:18 +0100, Timothy Redaelli wrote: cp -a "${S}"/* "${D}"/ || die "copying files failed!" >>> For future *BSD compatibility (yes i want to use the linux bsd emulation >>> for flash, opera, etc) it

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Mike Frysinger
On Monday 19 February 2007, Fabian Groffen wrote: > On 19-02-2007 14:14:18 +0100, Timothy Redaelli wrote: > > > cp -a "${S}"/* "${D}"/ || die "copying files failed!" > > > > For future *BSD compatibility (yes i want to use the linux bsd emulation > > for flash, opera, etc) it's better to use rsyn

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Fabian Groffen
On 19-02-2007 14:14:18 +0100, Timothy Redaelli wrote: > > cp -a "${S}"/* "${D}"/ || die "copying files failed!" > > For future *BSD compatibility (yes i want to use the linux bsd emulation > for flash, opera, etc) it's better to use rsync -a imho For my understanding, what's wrong with cp -pP

Re: [gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Timothy Redaelli
Simon Stelling wrote: > Hi all, > > I recently worked on the app-emulation/emul-linux-x86-* packages and > used almost identical code snipplets in them, so I realized I might as > well put together an eclass and drop the duplication. The eclass would > provide a common template for the following p

[gentoo-dev] New emul-libs.eclass

2007-02-19 Thread Simon Stelling
Hi all, I recently worked on the app-emulation/emul-linux-x86-* packages and used almost identical code snipplets in them, so I realized I might as well put together an eclass and drop the duplication. The eclass would provide a common template for the following packages: app-emulation/emul-