Re: [PATCH 2/3] dfa: new option DFA_STRAY_BACKSLASH_WARN

2022-06-04 Thread Paul Eggert
On 6/3/22 20:18, Bruno Haible wrote: This warning punishes a traditional habit, namely to backslash-escape a leading ASCII '-' character, so as to avoid it from being interpreted as an option. That's the first I've heard of that habit. Even 7th Edition Unix supported 'grep -e PAT FILE', but I

Re: Version sort behavior

2022-06-04 Thread Paul Eggert
On 6/3/22 20:48, Bruno Haible wrote: The coreutils test 'tests/misc/ls-misc.pl' now fails for me: It works for me, from a fresh checkout and bootstrap of the current coreutils commit 93e099e4c3b659b2e329f655fbdc73fdf594a66e on Fedora 36 x86-64. From the output, it looks like you might not

Re: [PATCH 2/3] dfa: new option DFA_STRAY_BACKSLASH_WARN

2022-06-04 Thread Bruno Haible
Paul Eggert wrote: > > This warning punishes a traditional habit, namely to backslash-escape > > a leading ASCII '-' character, so as to avoid it from being interpreted > > as an option. > > ... These days, although I would think 'grep -- PAT FILE' would work > everywhere Yes, these days "grep

Re: "grep '\]'" warnings suggest a Gnulib DFA patch

2022-06-04 Thread Paul Eggert
On 6/3/22 20:08, Bruno Haible wrote: But when I think about the thousands of people who use regular expressions out there. How would they remember that in parentheses both should be backslash-escaped in EREs \( \) It's even weirder, in that POSIX says unmatched ')' is treated like '\)'

Re: Version sort behavior

2022-06-04 Thread Bruno Haible
Paul Eggert wrote: > > The coreutils test 'tests/misc/ls-misc.pl' now fails for me: > > It works for me It works for me too now. I had done my 'git pull' one minute before you pushed the coreutils changes. Bruno