Re: iswblank failure

[Tom G. Christensen]

On Thu, Sep 02, 2010 at 03:21:46PM +0200, Ludovic Courtès wrote:
> Hello!
> 
> Bruno Haible <address@hidden> writes:
> 
> > 2010-09-02  Bruno Haible  <address@hidden>
> >
> >     wctype: Avoid compilation error on IRIX 6.5.30.
> >     * lib/wctype.in.h (iswblank): Declare with a replacement if
> >     REPLACE_ISWBLANK is set.
> >     * m4/wctype_h.m4 (gl_WCTYPE_H): Check also whether iswblank is
> >     declared. Set REPLACE_ISWBLANK.
> >     * modules/wctype (Makefile.am): Substitute REPLACE_ISWBLANK.
> >     * doc/posix-functions/iswblank.texi: Mention the IRIX 6.5.30 problem.
> >     * doc/posix-headers/wctype.texi: Likewise.
> >     Reported by Tom G. Christensen <address@hidden>.
> 
> This change causes problems:
> 
>   Inetutils, http://hydra.nixos.org/build/617935
>   libunistring, http://hydra.nixos.org/build/618008
>   Octave, http://hydra.nixos.org/build/618125
>   ...
> 
> The error is:
> 
> --8<---------------cut here---------------start------------->8---
> ./wctype.h:581:14: error: operator '||' has no right operand
> ./wctype.h:654:6: error: #if with no expression
> --8<---------------cut here---------------end--------------->8---
> 
> My guess is that this is due to a typo: ‘gl_WCTYPE_H’ sets
> ‘REPLACE_HAVE_ISWBLANK’ instead of ‘REPLACE_ISWBLANK’.
> 
I can confirm that s/REPLACE_HAVE_ISWBLANK/REPLACE_ISWBLANK/ fixes the
build and solves the problem on IRIX.

-tgc