[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: -Wundef warning in striconveh.h
From: |
Ludovic Courtès |
Subject: |
Re: -Wundef warning in striconveh.h |
Date: |
Mon, 25 Apr 2011 23:35:28 +0200 |
User-agent: |
Gnus/5.110015 (No Gnus v0.15) Emacs/23.3 (gnu/linux) |
Hello,
Andy Wingo <address@hidden> writes:
> On Thu 17 Mar 2011 15:05, Eric Blake <address@hidden> writes:
>
>> On 03/17/2011 04:20 AM, Andy Wingo wrote:
>>> striconveh.h:22 and striconveh.h:34 use #if when they should use #ifdef,
>>> or something.
>>
>> Not a bug in gnulib,
>>
>>>
>>> Building Guile a contributor got this warning:
>>
>> but in guile. Gnulib specifically documents that -Wundef is
>> incompatible with gnulib modules, and that our coding style is
>> deliberate. -Wundef only catches portability problems for ancient
>> (non-C89) compilers, and gnulib requires C89 as a bare minimum. While
>> you are free to use it for the rest of your project, you need to avoid
>> it while compiling gnulib (or at least ignore the warnings and avoid
>> -Werror).
>
> This error did not occur while compiling gnulib (in the lib/) directory,
> but rather libguile, which included striconveh.h. I believe we already
> follow your suggestions for building gnulib itself.
[...]
> See also this thread also:
>
> http://thread.gmane.org/gmane.comp.lib.gnulib.bugs/24225
>
> Do you have a response to Ludovic's concerns in his second message?
Unfortunately <verify.h> now similarly prevents use of -Wundef by its
users (again this happens when compiling libguile; Gnulib itself is
compiled _without_ -Wundef):
verify.h:177:8: error: "HAVE_STATIC_ASSERT" is not defined [-Werror=undef]
I’m giving up on compiling Guile itself with -Wundef, but I’d prefer it
if Gnulib did not impose its policy on Guile.
Would you be willing to fix the problematic headers? I’d be happy to
help with that, as I did in the past.
Thanks,
Ludo’.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: -Wundef warning in striconveh.h,
Ludovic Courtès <=