On Thu, Aug 18, 2022 at 8:22 AM Paul Eggert <eggert at cs.ucla.edu>
wrote:>
> On 8/17/22 23:58, Khem Raj wrote:
> > rsync which add -pedantic-errors option to cflags during configure
>
> OK, that's the problem then. 'configure' should not use flags
like
> -pedantic-errors. This has long been a property of
'autoconf'-generated
> scripts.
>
> The simplest way to fix this would be for rsync to not add
> -pedantic-errors, as it's more trouble than it's worth. If rsync
must
> add -pedantic-errors for some reason, it should do so at the very end of
> 'configure', as in the attached untested patch, so as not to screw
up
> earlier 'configure' tests; and perhaps it'd be better for rsync
to also
> add -pedantic-errors only for GCC not Clang (since Clang is more likely
> to mutate further in this area).
>
> Really, though, omitting -pedantic-errors entirely is the way to go.
> -pedantic-errors should be enabled only for special purposes, by
> explicit request of the person running 'configure' and/or
'make'; it's
> not suitable as a default option.
I think the autoconf patch should be good on its own merits regardless
since I have not looked deeply but there were few other package
builds which started to succeed after this change in autoconf so I think
rsync is not the only candidate. What do you think ?
>
> Anyway, please give this rsync patch a try. I'll cc it to the rsync
> mailing list to give rsync developers a heads-up about the compatibility
> problem building rsync with Clang 15 (which luckily does not exist yet :-).
Thanks for the rsync patch Paul, I will give it a shot.