On Tue, Jan 22, 2008 at 02:38:27PM +0100, Jim Meyering wrote: > Daniel Veillard <veillard@xxxxxxxxxx> 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@xxxxxxxxxx | libxml GNOME XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/ -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list