1 |
On Sat, 2009-02-21 at 19:44 -0500, Mike Frysinger wrote: |
2 |
> On Saturday 21 February 2009 19:38:33 Mart Raudsepp wrote: |
3 |
> > On Sat, 2009-02-21 at 19:29 -0500, Mike Frysinger wrote: |
4 |
> > > On Saturday 21 February 2009 19:00:19 Mart Raudsepp wrote: |
5 |
> > > > On Sat, 2009-02-21 at 18:55 -0500, Mike Frysinger wrote: |
6 |
> > > > > On Saturday 21 February 2009 18:38:55 Ryan Hill wrote: |
7 |
> > > > > > On Sat, 21 Feb 2009 18:27:10 -0500 Mike Frysinger wrote: |
8 |
> > > > > > > looks like bash-4.0 has broken semicolon escaping in subshells. |
9 |
> > > > > > > this comes up when using find's -exec like we do in a few places |
10 |
> > > > > > > in eclasses: ls=$(find "$1" -name '*.po' -exec basename {} .po |
11 |
> > > > > > > \;); shift you can work around the issue in a couple of ways: - |
12 |
> > > > > > > quote the semicolon: |
13 |
> > > > > > > .... ';') |
14 |
> > > > > > > - use backticks |
15 |
> > > > > > > `find .... \;` |
16 |
> > > > > > > |
17 |
> > > > > > > i'll tweak the eclasses to use quoting for now |
18 |
> > > > > > |
19 |
> > > > > > is this a bug or broken on purpose? |
20 |
> > > > > |
21 |
> > > > > i say it's a bug, but i'm not the bash maintainer |
22 |
> > > > > |
23 |
> > > > > i imagine it's fall out from attempts to fix support for case |
24 |
> > > > > statements in subshells |
25 |
> > > > |
26 |
> > > > Then the bug should be fixed, instead of changing usage to something |
27 |
> > > > apparently less common, as the conversion could miss some. And more |
28 |
> > > > importantly users still want to use \; for find -exec ending on their |
29 |
> > > > command line and their very own scripts. |
30 |
> > > > And who knows how many shell scripts shipped by packages use the |
31 |
> > > > escaping method. |
32 |
> > > |
33 |
> > > i think you missed the entire point of this thread: there's a bug in |
34 |
> > > bash-4.0 that code is likely to hit. |
35 |
> > |
36 |
> > I think you missed the entire point of my reply. |
37 |
> > That bug should be fixed, not workarounds applied all over the tree, as |
38 |
> > users still want to be able to escape semi-colons. |
39 |
> |
40 |
> no one suggested doing any of this crap you're talking about. if you want to |
41 |
> get all retarded, dont install the masked ebuild. i gave a heads up to people |
42 |
> who might want to experiment so they wouldnt have to figure out weird errors. |
43 |
> in the mean time, i tweaked a few common files so people wouldnt hit errors |
44 |
> and could investigate even further. |
45 |
|
46 |
Perhaps you should actually state those intentions at the start instead |
47 |
of starting to rant out on people replying. |
48 |
Sounds good now that we actually know what the plan is. |
49 |
|
50 |
|
51 |
-- |
52 |
Mart Raudsepp |
53 |
Gentoo Developer |
54 |
Mail: leio@g.o |
55 |
Weblog: http://planet.gentoo.org/developers/leio |