libctf doesn't build with Solaris sed

Nick Alcock nick.alcock@oracle.com
Wed Oct 21 14:52:26 GMT 2020


On 21 Oct 2020, Simon Marchi uttered the following:

> On 2020-10-21 10:15 a.m., Nick Alcock wrote:
>> Curious! I tested this on Solaris: maybe it works with some versions of
>> Solaris sed and not others.
>> 
>> It looks like this regex is not matching:
>> 
>>     s@^ *\(ECTF_[^[:blank:],]*\),\{0,1\}[[:blank:]]*/\* \(.*\).  \*/$@_CTF_STR (\1, "\2")@;
>> 
>> IIRC, Solaris sed is so archaic that it doesn't understand [:blank:]. If
>> you replace it with [\t ] or someting, does it work better? (I really
>> should look into enabling my compile farm access so I could check this
>> myself!)
>
> I replaced the two [:blank:] with [\t ] and it didn't seem to change anything.

Ugh. I'll see about getting my compile farm access working and have a
look myself. Maybe it's <{>} (and the only use of \{0,1\} can easily be
replaced with ?... which is *also* a GNU extension.

Perhaps this...

s@^ *\(ECTF_[^\t ,]*\),,*[\t ]*/\* \(.*\).  \*/$@_CTF_STR (\1, "\2")@;

... maybe?


More information about the Binutils mailing list