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