|
From: | Paolo Bonzini |
Subject: | Re: r150960 changed ltmain.sh and broke the build |
Date: | Wed, 26 Aug 2009 20:41:57 +0200 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.1) Gecko/20090814 Fedora/3.0-2.6.b3.fc11 Lightning/1.0pre Thunderbird/3.0b3 |
How about Patterns should not include the separator (unless escaped), even as part of a character class. In conformance with Posix, the Cray `sed' rejects `s/[^/]*$//': use `s,[^/]*$,,'. Even escaped, patterns should not include separators that are also used as metacharacters. For example, GNU sed 3.02 rejects `s,x\{1\,\},,', and is used on MinGW. ?
Yes, that's nice, except I would remove "and is used on MinGW".By the way even sed 4.0.x has that interpretation. Another semantic change is s|a\|b|c| which is alternation up to sed 4.0.x included, but in sed 4.1.x and above.
The reason is that sed up to 4.0.x passes the backslash to the regex matcher (causing trouble with \,) while 4.1.x strips it.
Paolo
[Prev in Thread] | Current Thread | [Next in Thread] |