This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Ignore warning in string/tester.c.


On Tue, Jan 06, 2015 at 05:20:47PM +0000, Joseph Myers wrote:
> On Tue, 6 Jan 2015, OndÅej BÃlka wrote:
> 
> > On Mon, Dec 15, 2014 at 08:15:47PM +0100, Torvald Riegel wrote:
> > > This fixes a warning in a build using a fairly recent GCC.  AFAIK the
> > > warning/option is new and hasn't been offered by 4.9, hence the
> > > conditional.  OK?
> > 
> > I also got warning in debian gcc-4.9 so I commited following as obvious.
> 
> No, it's not obvious; as far as I can tell, the option is not in GCC 4.9 
> branch, so resulting in
> 
> tester.c: In function 'test_memset':
> tester.c:1311:3: error: unknown option after '#pragma GCC diagnostic' kind [-Werror=pragmas]
>    DIAG_IGNORE_NEEDS_COMMENT (4.9, "-Wmemset-transposed-args")
>    ^
> cc1: all warnings being treated as errors
> 
> and breaking the build.  Please revert the patch.  If distributions have 
> backported this warning, a configure test will be needed to determine 
> whether it can be suppressed.
> 
ok, however that is bad policy as you would need to add exception every
time some distribution decides to backport warning. I would rather
disable -Werror=pragmas.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]