[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Libvir] libvirt on OS X Leopard 10.5.1



On Tue, Jan 22, 2008 at 02:38:27PM +0100, Jim Meyering wrote:
> Daniel Veillard <veillard redhat com> wrote:
> 
> > On Tue, Jan 22, 2008 at 08:00:34AM -0500, Daniel Veillard wrote:
> >> On Mon, Jan 21, 2008 at 11:39:35PM +0000, Daniel P. Berrange wrote:
> >> > I would point you at the snapshot tar.gz here, but it appears to be not
> >> > updated since September
> >> >
> >> >   http://libvirt.org/downloads.html
> >>
> >> ./configure: line 35987: syntax error near unexpected token `libxml-2.0,LIBXML_FOUND=yes'
> >> ./configure: line 35987: `    PKG_CHECK_EXISTS(libxml-2.0,LIBXML_FOUND=yes)'
> >>
> >>  libvirt.org is a RHEL4 box, seems the pkgconfig requirements which were
> >> introduced break running autogen.sh , so no Makefile, make dist, and no
> >> snapshot apparently. I will try to investigate but I can't believe we managed
> >> to break detection for something as common as libxml2 *on xmlsoft.org itself*,
> >> the irony ...
> >
> >   Upgrading pkg-config on the box this goes further but then gets stuck
> > at:
> >
> > configure: error: conditional "HAVE_SASL" was never defined.
> > Usually this means the macro was only invoked conditionally.
> >
> > trying to autogen with
> >   ./autogen.sh --prefix=/usr --without-depends --without-sasl
> >
> > I don't understand the problem, it seems
> >   AM_CONDITIONAL(HAVE_SASL, [test "$with_sasl" != "no"])
> > is set outside of any conditional block in configure.in
> 
> Right.
> Here's an untested patch:
[...]
>  dnl Cyrus SASL
> +have_sasl=0
>  AC_ARG_WITH(sasl,
>    [  --with-sasl         use cyrus SASL for authentication],
>    [],
> @@ -402,8 +403,10 @@ if test "$with_sasl" != "no"; then
>    CFLAGS="$old_cflags"
>    LIBS="$old_libs"
>    SASL_LIBS="$SASL_LIBS -lsasl2"
> -  AC_DEFINE_UNQUOTED(HAVE_SASL, 1, [whether Cyrus SASL is available for authentication])
> +  have_sasl=1
>  fi
> +AC_DEFINE_UNQUOTED(HAVE_SASL, $have_sasl,
> +		   [whether Cyrus SASL is available for authentication])
>  AM_CONDITIONAL(HAVE_SASL, [test "$with_sasl" != "no"])
>  AC_SUBST(SASL_CFLAGS)
>  AC_SUBST(SASL_LIBS)

  Thanks Jim, looking at the patch I think I understand the error message
now, but I applied the patch (or rather made the change manually), and
I still get the same:
-----
xmlsoft:~/libvirt -> ./autogen.sh --prefix=/usr --without-depends --without-sasl
[no error reported until ...]
checking whether NLS is requested... yes
checking for GNU gettext in libc... yes
checking whether to use NLS... yes
checking where the gettext function comes from... libc
configure: error: conditional "HAVE_SASL" was never defined.
Usually this means the macro was only invoked conditionally.
xmlsoft:~/libvirt ->
-----

also I'm not sure, shouldn't the AM_CONDITIONAL test check $have_sasl instead
or $with_sasl ?

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
veillard redhat com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]