Re: [PATCH] maint.mk: suppress a syntax check false positive

2011-03-11 Thread Jim Meyering
Pádraig Brady wrote: > A recent change to dd in coreutils triggered this issue, > which is avoided with the attached. Thanks! I've pushed that. > + maint.mk: suppress a false positive warning > + * top/maint.mk ((sc_unmarked_diagnostics): Don't warn when > + diagnostics are marked wit

Re: [PATCH] maint.mk: suppress a syntax check false positive

2011-03-11 Thread Pádraig Brady
On 11/03/11 09:32, Jim Meyering wrote: > Pádraig Brady wrote: >> A recent change to dd in coreutils triggered this issue, >> which is avoided with the attached. > > Thanks! > I've pushed that. Cool, thanks. I have gnulib commit access BTW I had a local tweak to remove an extraneous ( in the Chang

Re: [PATCH] maint.mk: suppress a syntax check false positive

2011-03-11 Thread Jim Meyering
Pádraig Brady wrote: > On 11/03/11 09:32, Jim Meyering wrote: >> Pádraig Brady wrote: >>> A recent change to dd in coreutils triggered this issue, >>> which is avoided with the attached. >> >> Thanks! >> I've pushed that. > > Cool, thanks. > I have gnulib commit access BTW > I had a local tweak to

Re: bug#8230: touch dumps core on solaris 10

2011-03-11 Thread Eric Blake
[adding bug-gnulib] On 03/10/2011 06:53 PM, Ben Walton wrote: > > Hi All, > > Since 8.8, touch has dumped core on solaris 10 (built on solaris 9). > The command that triggers the core dump is a simple: touch somefile > > This is seemingly related to the recommend patch cluster of Jan 10, > 2011

Re: [PATCH] maint.mk: suppress a syntax check false positive

2011-03-11 Thread Eric Blake
On 03/11/2011 02:32 AM, Jim Meyering wrote: > Pádraig Brady wrote: >> A recent change to dd in coreutils triggered this issue, >> which is avoided with the attached. > >> sc_unmarked_diagnostics: >> @grep -nE \ >> '\> - | gr

Re: [PATCH] maint.mk: suppress a syntax check false positive

2011-03-11 Thread Pádraig Brady
On 11/03/11 13:50, Eric Blake wrote: > On 03/11/2011 02:32 AM, Jim Meyering wrote: >> Pádraig Brady wrote: >>> A recent change to dd in coreutils triggered this issue, >>> which is avoided with the attached. >> >>> sc_unmarked_diagnostics: >>> @grep -nE

Re: bug#8230: touch dumps core on solaris 10

2011-03-11 Thread Bruno Haible
Regarding : Eric Blake wrote: > If you compiled with debugging, then it might help to show a gdb session > of rpl_fcntl in lib/fcntl.c while stepping enough to prove the loop, as > well as a backtrace that leads to that loop, might be worthwhile.

Re: bug#8230: touch dumps core on solaris 10

2011-03-11 Thread Paul Eggert
On 03/11/2011 02:41 PM, Bruno Haible wrote: > I propose to change gnulib so that it uses > #define futimens rpl_futimens > #define utimensat rpl_utimensat > also on Solaris 9 (with the usual idioms for the C++ and the gnulib > namespace). That sounds good, but why make Solaris 9 a special cas