Re: [PATCH 08/89] configure: use LIBVIRT_ARG_WITH(_ALT) macros

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

 



On Mon, Dec 19, 2016 at 15:10:01 +0100, Pavel Hrdina wrote:
> On Mon, Dec 19, 2016 at 02:16:38PM +0100, Jiri Denemark wrote:
> > On Fri, Dec 16, 2016 at 10:10:36 +0100, Pavel Hrdina wrote:
> > > Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
...
> > This hunk doesn't seem to be correct. First, "x$with_libxml" != "xno"
> > is always true in the else branch of "x$with_libxml" = "xno". But more
> > importantly, you would remove the option to override the path to libxml
> > using --with-libxml=/path/to/libxml in case another libxml library is
> > installed.
> 
> That's true, this is just left over after splitting the changes into separate
> commits.  I'll update it to this:
> 
> @@ -1033,16 +974,16 @@ LIBXML_CFLAGS=""
>  LIBXML_LIBS=""
>  LIBXML_FOUND="no"
>  
> -AC_ARG_WITH([libxml], [AS_HELP_STRING([--with-libxml=@<:@PFX@:>@],
> -   [libxml2 location])])
> +LIBVIRT_ARG_WITH_ALT([LIBXML], [libxml2 (>= 2.6.0) location], [check])
> +
>  if test "x$with_libxml" = "xno" ; then
>      AC_MSG_CHECKING(for libxml2 libraries >= $LIBXML_REQUIRED)
>      AC_MSG_ERROR([libxml2 >= $LIBXML_REQUIRED is required for libvirt])
> -elif test "x$with_libxml" = "x" && test "x$PKG_CONFIG" != "x" ; then
> +elif test "x$with_libxml" = "xcheck" && test "x$PKG_CONFIG" != "x" ; then
>      PKG_CHECK_MODULES(LIBXML, libxml-2.0 >= $LIBXML_REQUIRED, [LIBXML_FOUND=yes], [L
> IBXML_FOUND=no])
>  fi
>  if test "$LIBXML_FOUND" = "no" ; then
> -    if test "x$with_libxml" != "x" ; then
> +    if test "x$with_libxml" != "xcheck" ; then
>         LIBXML_CONFIG=$with_libxml/bin/$LIBXML_CONFIG
>      fi
>      AC_MSG_CHECKING(libxml2 $LIBXML_CONFIG >= $LIBXML_REQUIRED )

ACK

> > > +LIBVIRT_ARG_WITH_ALT([QEMU_USER], [username to run QEMU system instance as],
> > > +                     ['platform dependent'])
> > 
> > Any reason why $default_qemu_user is not used as the default value
> > rather than 'platform dependent'?
> 
> I was not able to achieve it after a day of reading documentation and autoconf
> code and googling.  It would be better, but I don't know how and if it's even
> possible.

And even if you managed to do so, it would really make a lot of sense
since $default_qemu_user is based on $with_freebsd and $with_osx which
will not be properly set when running configure --help.

Jirka

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux