"Roy T. Fielding" <[EMAIL PROTECTED]> 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 | [EMAIL PROTECTED] | PGP public key at web site: http://www.geocities.com/SiliconValley/Park/9289/ Born in Roswell... married an alien...