Re: [xml] minor compile warnings

[Roumen Petrov <bugtrack roumenpetrov info>]

Daniel Veillard wrote:
> On Tue, Feb 05, 2008 at 11:50:23PM +0200, Roumen Petrov wrote:
>   
> > [SNIP]
> >     
> > > AM_ICONV is defined in iconv.m4. I can found it in gettext-0.10.38 
> > > (gnu site begin with gettext-0.10.40 from 15-Sep-2001).
> > > The macro use AC_DEFINE_UNQUOTED, AC_TRY_COMPILE, AC_TRY_LINK.
> > > I can found them in autoconf 2.50 but I'm not sure for 2.13.
> > >
> > > So that for AM_ICONV: AC_PREREQ(2.50) is sufficient.
> > >       
> > Just found that configure scrip for openldap 2.1.x use 
> > AC_DEFINE_UNQUOTED, AC_TRY_COMPILE, AC_TRY_LINK and is with 
> > AC_PREREQ(2.13) so autoconf 2.13 is sufficient.
> >     
>
>   Okay, I really dislike auto* hacking and debugging, so if you
> provide a patch, I will apply it,
>
>   thanks,
>
> Daniel
>
>   

Unfortunately AM_ICONV from iconv.m4 version AM4(from gettext 0.11 or 
subsequent) introduce dependency on more lib-xxx.m4 macro files and will 
change libxml2 configure options(arguments).

If one day libxml2 go international and decide to use gettext system for 
message translation existing check for iconv can be replaced with macro 
AM_ICONV.

I agree that this is just warning and may left it as is.

To email is attached file "libxml2-trunk-iconvconst.patch.gz" that mimic 
check for iconv second argument but without new iconv.m4 complexity.


Regards,
Roumen

Attachment: libxml2-trunk-iconvconst.patch.gz
Description: application/gzip