On Mon, Nov 05, 2012 at 10:11:17AM +0000, Richard W.M. Jones wrote: > On Mon, Nov 05, 2012 at 01:23:51PM +0800, Gao feng wrote: > > +dnl libfuse > > +AC_ARG_WITH([fuse], > > + AC_HELP_STRING([--with-fuse], [use libfuse to proivde fuse filesystem support for libvirt lxc]), > > + [], > > + [with_fuse=check]) > > +dnl > > +dnl This check looks for 'fuse' > > +dnl > > +FUSE_CFLAGS= > > +FUSE_LIBS= > > +if test "x$with_fuse" != "xno"; then > > + PKG_CHECK_MODULES([FUSE], [fuse >= $FUSE_REQUIRED], > > + [with_fuse=yes], [ > > + if test "x$with_fuse" = "xcheck" ; then > > + with_fuse=no > > + else > > + AC_MSG_ERROR( > > + [You must install fuse Library to compile libvirt]) > > + fi > > + ]) > > + if test "x$with_fuse" = "xyes" ; then > > + FUSE_LIBS="-lfuse" > > + FUSE_CFLAGS="-D_FILE_OFFSET_BITS=64" As per Rich's comments, these 2 lines are bogus. The PKG_CHECK_MODULES macro should already be setting FUSE_LIBS and FUSE_CFLAGS to the correct values defined in the pkg-config file for FUSE. > > + AC_DEFINE_UNQUOTED([HAVE_FUSE], 1, [whether fuse is available for libvirt lxc]) > > + fi > > +fi > > +AM_CONDITIONAL([HAVE_FUSE], [test "x$with_fuse" = "xyes"]) > > +AC_SUBST([FUSE_CFLAGS]) > > +AC_SUBST([FUSE_LIBS]) > > + > > For comparison, here is how we test for fuse in libguestfs: > > dnl FUSE is optional to build the FUSE module. > AC_ARG_ENABLE([fuse], > AS_HELP_STRING([--disable-fuse], [disable FUSE (guestmount) support]), > [], > [enable_fuse=yes]) > AS_IF([test "x$enable_fuse" != "xno"], > [PKG_CHECK_MODULES([FUSE],[fuse], > [AC_SUBST([FUSE_CFLAGS]) > AC_SUBST([FUSE_LIBS]) > AC_DEFINE([HAVE_FUSE],[1],[Define to 1 if you have FUSE.]) > old_LIBS="$LIBS" > LIBS="$FUSE_LIBS $LIBS" > AC_CHECK_FUNCS([fuse_opt_add_opt_escaped]) > LIBS="$old_LIBS" > ], > [enable_fuse=no > AC_MSG_WARN([FUSE library and headers are missing, so optional FUSE module won't be built]) > ]) > ]) > AM_CONDITIONAL([HAVE_FUSE],[test "x$enable_fuse" != "xno"]) > > [ https://github.com/libguestfs/libguestfs/blob/master/configure.ac#L744 ] > > Is there any reason not to use PKG_CHECK_MODULES? Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list