Re: [PATCH] maintainer-makefile: Fix Apple Xcode 'make syntax-check'.

2022-11-15 Thread Bruno Haible
Pádraig Brady wrote: > > +sc_unportable_grep_q: > > + @prohibit='grep -q' halt="unportable 'grep -q', use >/dev/null instead" > \ > > +$(_sc_search_regexp) > > + > > Note the above isn't equivalent as -q will exit early upon first match > (as defined by POSIX) In most of the cases, the

Re: [PATCH] maintainer-makefile: Fix Apple Xcode 'make syntax-check'.

2022-11-15 Thread Pádraig Brady
On 01/11/2022 08:12, Simon Josefsson via Gnulib discussion list wrote: Bruno Haible writes: Hi Simon, + @if ! indent --version 2> /dev/null | grep -q 'GNU indent'; then\ As mentioned in the Autoconf manual [1], the grep option '-q' is not portable. E.g. on Solaris 10: $ echo | grep

Re: [PATCH] maintainer-makefile: Fix Apple Xcode 'make syntax-check'.

2022-11-01 Thread Simon Josefsson via Gnulib discussion list
Bruno Haible writes: > Hi Simon, > >> +@if ! indent --version 2> /dev/null | grep -q 'GNU indent'; then\ > > As mentioned in the Autoconf manual [1], the grep option '-q' is not portable. > E.g. on Solaris 10: > > $ echo | grep -q x > grep: illegal option -- q > Usage: grep -hblcnsviw pattern

Re: [PATCH] maintainer-makefile: Fix Apple Xcode 'make syntax-check'.

2022-10-31 Thread Bruno Haible
Hi Simon, > + @if ! indent --version 2> /dev/null | grep -q 'GNU indent'; then\ As mentioned in the Autoconf manual [1], the grep option '-q' is not portable. E.g. on Solaris 10: $ echo | grep -q x grep: illegal option -- q Usage: grep -hblcnsviw pattern file . . . The portable alternative