From: | Roy Marples <uberlord@g.o> | ||
---|---|---|---|
To: | gentoo-dev@l.g.o | ||
Subject: | Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in app-misc/note: ChangeLog note-1.3.3.ebuild | ||
Date: | Mon, 15 Oct 2007 08:48:27 | ||
Message-Id: | 1192437335.1277.1.camel@uberlaptop.development.ltl | ||
In Reply to: | Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in app-misc/note: ChangeLog note-1.3.3.ebuild by "Bo Ørsted Andresen" |
1 | On Mon, 2007-10-15 at 09:56 +0200, Bo Ørsted Andresen wrote: |
2 | > On Monday 15 October 2007 09:42:50 Donnie Berkholz wrote: |
3 | > > On 07:40 Mon 15 Oct , Christian Faulhammer (opfer) wrote: |
4 | > > > 1.1 app-misc/note/note-1.3.3.ebuild |
5 | > > > |
6 | > > > file : http://sources.gentoo.org/viewcvs.py/gentoo-x86/app-misc/note/note-1.3.3.ebuild?rev=1.1&view=markup |
7 | > > > plain: http://sources.gentoo.org/viewcvs.py/gentoo-x86/app-misc/note/note-1.3.3.ebuild?rev=1.1&content-type=text/plain |
8 | > > > |
9 | > > > for v in mysql text dbm general; do |
10 | > > > if ! use ${v}; then |
11 | > > > for u in `find "${D}" -type f -name *${v}.*pm`; do |
12 | > > > rm ${u} |
13 | > > |
14 | > > Looks like 'u' could have spaces in it, since it's based on D. |
15 | > |
16 | > Written like this u certainly can't since the output of the find expression is |
17 | > subjected to word splitting before u gets set. Hence this will fail if ${D} |
18 | > contains spaces even if ${u} gets quoted. The best way to solve this is to use |
19 | > find ... -print0 | xargs -0 rm. Unfortunately that solution means using GNU |
20 | > extensions that I'm not sure are guaranteed to be available in the ebuild |
21 | > environment. If they aren't maybe they should be. |
22 | > |
23 | |
24 | find "${D}" -type f -name *${v}.*pm -delete |
25 | |
26 | Thanks |
27 | |
28 | Roy |
29 | |
30 | -- |
31 | gentoo-dev@g.o mailing list |
Subject | Author |
---|---|
Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in app-misc/note: ChangeLog note-1.3.3.ebuild | Fabian Groffen <grobian@g.o> |