apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Trawick <trawi...@bellsouth.net>
Subject Re: [PATCH] a somewhat different approach to the iconv() issue
Date Thu, 08 Feb 2001 19:53:45 GMT
"Roy T. Fielding" <fielding@ebuilt.com> writes:

> > > What was checked in doesn't work -- I now get warnings again on
> > > my platform, and mine is the one with the correct prototypes.
> > 
> > Hmmm...  I get no warnings on a system of each flavor (Tru64 and
> > RedHat 6) and got no warnings.  What warnings do you get?  Did the
> > configure output display the right message?
> > 
> > checking for type of inbuf parameter to iconv... char **
> 
> RedHat 7 with the latest glibc compiled from source.  There are
> two problems, one easy and one not yet figured out:
> 
>   1) The #ifdef APR_ in xlate.c must be a #if

Why is that?  We generally use #ifdef FOO if FOO is sometimes defined
and sometimes not (e.g., APR_ICONV_INBUF_CONST).  We generally use #if
FOO if FOO is always defined but sometimes to 1 and sometimes to 0
(e.g., APR_HAS_XLATE).

>   2) The test is looking at the versions and triggering #error
> 
> I don't know why the version test is giving a false-positive, nor
> do I particularly care -- I don't think this can be fixed by assuming
> the version numbers mean anything.  The fix I am working on is a
> replacement for AC_TRY_COMPILE, but m4 isn't one of my strong
> points.

I don't follow your comment about "assuming the version numbers mean
anything".  Maybe if I saw the version numbers in glibc 2.2's
<features.h> it would be easy to follow what you are saying.

If you can get -Werror working for gcc, obviously we don't need the
glibc version check since we can expect to use gcc on a platform which
uses glibc 99.99% of the time, but I can't help but be curious about
why it generated the #error on RH 7.0 which has glibc 2.2 (and I guess
__GLIBC_MINOR__ == 2).

(I wasn't too eager to write the compile macro either.  When I
considered a different flavor of AC_TRY_COMPILE it was obvious that I
would have to start by cutting and pasting AC_TRY_COMPILE.  A quick
look at the license reminded me that I am not a lawyer :) )

-- 
Jeff Trawick | trawickj@bellsouth.net | PGP public key at web site:
       http://www.geocities.com/SiliconValley/Park/9289/
             Born in Roswell... married an alien...

Mime
View raw message