On Thu, Aug 18, 2022 at 8:22 AM Paul Eggert <[email protected]> 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.
