Alistair Francis writes ("Re: [Xen-devel] [PATCH v2 1/5] Remove hardcoded strict -Werror checking"): > On Thu, Dec 22, 2016 at 12:41 AM, Jan Beulich <jbeul...@suse.com> wrote: >> On 20.12.16 at 20:46, <alistair.fran...@xilinx.com> wrote: > >> Signed-off-by: Alistair Francis <alistair.fran...@xilinx.com> > > > > Without some rationale given I don't think such changes are > > acceptable at all. And then, as already pointed out others, the > > use of -Werror is there not just for fun. If anything I think an > > override to that default could be acceptable. > > Unfortunately the APPEND_CFLAGS=-Wno-error doesn't fix all the issues > as I still see warnings/errors when building: tools/kconfig/conf.c.
That sounds like a bug to me. Do you know why it's not effective there ? > Everyone seems fairly open to an override. Is a environment variable, > which if set will disable Werror acceptable? Something like NO_ERROR=Y > which will result in no -Werror being appended. Yes, an environment variable would be acceptable, but it should have the right name and semantics and ideally we could reuse an existing variable or fix it if it is broken. How about `APPEND_CFLAGS=-Wno-error' ? :-) Thanks, Ian. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel