[Top][All Lists]

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

Re: [libvirt] [PATCH] autogen: added glibtool and pkg-config checks

From: Eric Blake
Subject: Re: [libvirt] [PATCH] autogen: added glibtool and pkg-config checks
Date: Wed, 15 Sep 2010 11:02:20 -0600
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv: Gecko/20100907 Fedora/3.1.3-1.fc13 Mnenhy/0.8.3 Thunderbird/3.1.3

[adding bug-gnulib]

On 09/15/2010 06:36 AM, Justin Clift wrote:
OSX MacPorts has libtool named as glibtool, with libtoolize
named as glibtoolize.  This patch adds support for this, and also
adds a check for pkg-config, to warn when it is missing.

Hmm - would it be easier to make bootstrap.conf list pkg-config as a prerequisite, rather than having to hack up autogen.sh to do that?

Also, if I'm reading the code right, changing bootstrap.conf to list 'libtoolize' instead of 'libtool' as the prerequisite will let you get by with overriding just $LIBTOOLIZE instead of both $LIBTOOL and $LIBTOOLIZE.

Additionally it may be sufficient to just teach upstream gnulib/build-aux/bootstrap to use 'find_tool LIBTOOLIZE libtoolize glibtoolize', at which point re-syncing to upstream bootstrap will automatically pick up on the right libtool for MacOS without you having to hack libvirt's autogen.sh in the first place :)

So, NACK to this version of the libvirt patch, and instead let me do some gnulib work... Not to mention that Gary is working on some upstream patches to improve bootstrap modularity, and there is also a request to add 'bootstrap --skip-git' that I need to respond to...

Eric Blake   address@hidden    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

reply via email to

[Prev in Thread] Current Thread [Next in Thread]