Re: [Bug-wget] [Patch] Use -isystem in Makefile to suppress warnings from libraries

[Darshit Shah]

For example, variable length arrays and the "inline" keyword are both
flagged by the compiler as an extension and gnulib and gettext bit use
them heavily. Without this patch, my builds on travis are failing
because the output logs are > 4MB.

At the risk of starting another fight, can we move to C99 as the
standard? The only major compiler I know of that doesn't support it is
MSVC. But do we really need to stay so much behind times to support
MSVC? Windows users can use CygWin / MingW to compile Wget. Half of
gnulib, gettext or other libraries will probably not compile properly
using MSVC anyways. Microsoft is not keen on improving their support
for C runtimes.

Is there anything else that I'm missing?

On 29 January 2016 at 15:45, Darshit Shah <address@hidden> wrote:
> Most of them are actually false positives, probably due to us. Gnulib
> uses some more modern code extensions and the compiler keeps warning
> us about it since we set the C language to std=gnu89. I'm not happy
> about this fact, but this discussion has happened multiple times and I
> don't think we will be moving to a more modern setup anytime soon. I
> would personally prefer using *at least* C99, a more recent version
> like C11 would be even better, but not all compiler would support
> that.
>
> On 29 January 2016 at 15:40, Eli Zaretskii <address@hidden> wrote:
>>> From: Darshit Shah <address@hidden>
>>> Date: Fri, 29 Jan 2016 15:18:57 +0100
>>>
>>> A recent GCC / LLVM update has caused my setup to spew far too many
>>> warnings on compiling Wget. On a closer look, they all come from
>>> Gnulib code. I propose the attached patch to explicitly mark those
>>> files as libraries and have the compiler suppress warnings from them.
>>> This way we can focus on the warnings generated by Wget codebase
>>> alone.
>>
>> If we do this, who will tell Gnulib people to get their act together
>> and fix those warnings?  I think the right solution to this is in
>> Gnulib, not in Wget.
>>
>> Thanks.
>
>
>
> --
> Thanking You,
> Darshit Shah



-- 
Thanking You,
Darshit Shah