On Tue, Jun 18, 2019 at 05:39:43PM +0300, Tuomo Soini wrote: > On Tue, 18 Jun 2019 15:34:00 +0200 > Antony Antony <[email protected]> wrote: > > > so based on this long discussions the final proposal is: > > > > diff --git a/mk/userland-cflags.mk b/mk/userland-cflags.mk > > -WERROR_CFLAGS?=-Werror > > +WERROR_CFLAGS?=-Werror -Wno-missing-field-initializers > > > > As long as we want to support CentOS6 or older compilers we should > > add this. I know it works on CentOS 6 and clang. Currently CentOS 6 > > is broken, Travis tests, due to the fowling error. > > > > Tumo's work around is WERROR_CFLAGS='' He is pushing this for > > travis, CentOS 6 testing too. I hopping there is a better fix. > > No. WERROR_CFLAGS= is not needed on spec because > -Wno-missing-field-initializers was already there, in USER_CFLAGS
this thread is proposing to make "-Wno-missing-field-initializers" default, in userland-cflags.mk. All compilers can take it, not just CentOS 6 or old ones. And it seems good for ppc too. If something can be global why put it in a distribution specific, spec, file. I am going to push it soon. -antony _______________________________________________ Swan-dev mailing list [email protected] https://lists.libreswan.org/mailman/listinfo/swan-dev
