autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: r150960 changed ltmain.sh and broke the build


From: Dave Korn
Subject: Re: r150960 changed ltmain.sh and broke the build
Date: Wed, 26 Aug 2009 20:08:29 +0100
User-agent: Thunderbird 2.0.0.17 (Windows/20080914)

Ralf Wildenhues wrote:
> [ trimmed Cc:, added autoconf-patches; followups can remove gcc@ ]
> [ http://thread.gmane.org/gmane.comp.gcc.devel/108348 ]
> 
> * Dave Korn wrote on Wed, Aug 26, 2009 at 08:16:11PM CEST:
>> Ralf Wildenhues wrote:
>>>      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,[^/]*$,,'.
>>> [...]
>>>      Portable `sed' regular expressions should use `\' only to escape
>>>      characters in the string `$()*.0123456789[\^n{}'.
>>   Ah, I didn't read those two conditions as applying simultaneously.
> 
> I don't think they apply simultaneously in the way you interpreted that.
> However, in the  s,x\{1\,\},y,  example, the comma is both a separator
> and another metacharacter.

  That seems strange to me, are you saying the comma is a metachar because it
is a component of the curly-brace number-of-repetitions modifier?

>  Hmm, maybe those sentences don't really cover this case.
> 
> 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.

  Well, modulo my surprise that a comma is considered a metachar, that makes
sense.

    cheers,
      DaveK





reply via email to

[Prev in Thread] Current Thread [Next in Thread]