tests/lexical.at: fix regexp for backslash quoting tests for newer 'sed'

tcsh-org / tcsh

This is a read-only mirror of the tcsh code repository.

https://www.tcsh.org/

Other

237 stars 42 forks source link

tests/lexical.at: fix regexp for backslash quoting tests for newer 'sed' #1

Closed deekej closed 6 years ago

deekej commented 7 years ago

During the Fedora 26 Mass Rebuild, it was discovered build of tcsh would fail in the make check phase. Further investigation showed me that sed was failing with this error message:

sed: character class syntax is [[:space:]], not [:space:]

Apparently, starting with version 4.3 (or newer version used in Fedora), sed no longer supports using simple [:group:]*. Instead, the [[:group:]]* has to be used.

deekej commented 7 years ago

Had anybody chance to look into this, please? :)

zoulasc commented 7 years ago

On Feb 23, 8:04am, notifications@github.com (deekej) wrote: -- Subject: Re: [tcsh-org/tcsh] tests/lexical.at: fix regexp for backslash qu

| Had anybody chance to look into this, please? :)

Yes, Looks good and I will commit it.

christos