From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:43562) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Quu9R-0000b6-Bo for qemu-devel@nongnu.org; Sat, 20 Aug 2011 18:36:26 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Quu9Q-0005zl-3z for qemu-devel@nongnu.org; Sat, 20 Aug 2011 18:36:25 -0400 Received: from speedy.comstyle.com ([206.51.28.2]:3491 helo=mail.comstyle.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Quu9Q-0005zf-05 for qemu-devel@nongnu.org; Sat, 20 Aug 2011 18:36:24 -0400 Message-ID: <4E5036E0.7030100@comstyle.com> Date: Sat, 20 Aug 2011 18:36:16 -0400 From: Brad MIME-Version: 1.0 References: <20110814003014.GA8665@rox.home.comstyle.com> In-Reply-To: <20110814003014.GA8665@rox.home.comstyle.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] Improvements to libtool support. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Alon Levy On 13/08/11 8:30 PM, Brad wrote: > Improvements to the libtool support in QEMU. Replace hard coded > libtool in the infrastructure with $(LIBTOOL) and allow > overriding the libtool binary used via the configure > script. > > Signed-off-by: Brad Smith Alon? > --- > Makefile.objs | 2 +- > configure | 5 ++--- > libcacard/Makefile | 10 +++++----- > rules.mak | 2 +- > 4 files changed, 9 insertions(+), 10 deletions(-) > > diff --git a/Makefile.objs b/Makefile.objs > index 16eef38..6c0be08 100644 > --- a/Makefile.objs > +++ b/Makefile.objs > @@ -369,7 +369,7 @@ trace-dtrace.lo: trace-dtrace.dtrace > @echo "missing libtool. please install and rerun configure."; exit 1 > else > trace-dtrace.lo: trace-dtrace.dtrace > - $(call quiet-command,libtool --mode=compile --tag=CC dtrace -o $@ -G -s $<, " lt GEN trace-dtrace.o") > + $(call quiet-command,$(LIBTOOL) --mode=compile --tag=CC dtrace -o $@ -G -s $<, " lt GEN trace-dtrace.o") > endif > > simpletrace.o: simpletrace.c $(GENERATED_HEADERS) > diff --git a/configure b/configure > index 6c77067..eb7497b 100755 > --- a/configure > +++ b/configure > @@ -224,6 +224,7 @@ cc="${CC-${cross_prefix}gcc}" > ar="${AR-${cross_prefix}ar}" > objcopy="${OBJCOPY-${cross_prefix}objcopy}" > ld="${LD-${cross_prefix}ld}" > +libtool="${LIBTOOL-${cross_prefix}libtool}" > strip="${STRIP-${cross_prefix}strip}" > windres="${WINDRES-${cross_prefix}windres}" > pkg_config="${PKG_CONFIG-${cross_prefix}pkg-config}" > @@ -1341,10 +1342,8 @@ fi > ########################################## > # libtool probe > > -if ! has libtool; then > +if ! has $libtool; then > libtool= > -else > - libtool=libtool > fi > > ########################################## > diff --git a/libcacard/Makefile b/libcacard/Makefile > index fe9747a..56dc974 100644 > --- a/libcacard/Makefile > +++ b/libcacard/Makefile > @@ -37,7 +37,7 @@ install-libcacard: > @echo "libtool is missing, please install and rerun configure"; exit 1 > else > libcacard.la: $(libcacard.lib-y) $(QEMU_OBJS_LIB) > - $(call quiet-command,libtool --mode=link --quiet --tag=CC $(CC) $(libcacard_libs) -lrt -rpath $(libdir) -o $@ $^," lt LINK $@") > + $(call quiet-command,$(LIBTOOL) --mode=link --quiet --tag=CC $(CC) $(libcacard_libs) -lrt -rpath $(libdir) -o $@ $^," lt LINK $@") > > libcacard.pc: $(libcacard_srcpath)/libcacard.pc.in > sed -e 's|@LIBDIR@|$(libdir)|' \ > @@ -53,10 +53,10 @@ install-libcacard: libcacard.pc libcacard.la vscclient > $(INSTALL_DIR) "$(DESTDIR)$(libdir)/pkgconfig" > $(INSTALL_DIR) "$(DESTDIR)$(libcacard_includedir)" > $(INSTALL_DIR) "$(DESTDIR)$(bindir)" > - libtool --mode=install $(INSTALL_PROG) vscclient "$(DESTDIR)$(bindir)" > - libtool --mode=install $(INSTALL_DATA) libcacard.la "$(DESTDIR)$(libdir)" > - libtool --mode=install $(INSTALL_DATA) libcacard.pc "$(DESTDIR)$(libdir)/pkgconfig" > + $(LIBTOOL) --mode=install $(INSTALL_PROG) vscclient "$(DESTDIR)$(bindir)" > + $(LIBTOOL) --mode=install $(INSTALL_DATA) libcacard.la "$(DESTDIR)$(libdir)" > + $(LIBTOOL) --mode=install $(INSTALL_DATA) libcacard.pc "$(DESTDIR)$(libdir)/pkgconfig" > for inc in *.h; do \ > - libtool --mode=install $(INSTALL_DATA) $(libcacard_srcpath)/$$inc "$(DESTDIR)$(libcacard_includedir)"; \ > + $(LIBTOOL) --mode=install $(INSTALL_DATA) $(libcacard_srcpath)/$$inc "$(DESTDIR)$(libcacard_includedir)"; \ > done > endif > diff --git a/rules.mak b/rules.mak > index 1a2622c..884d421 100644 > --- a/rules.mak > +++ b/rules.mak > @@ -22,7 +22,7 @@ ifeq ($(LIBTOOL),) > @echo "missing libtool. please install and rerun configure"; exit 1 > else > %.lo: %.c > - $(call quiet-command,libtool --mode=compile --quiet --tag=CC $(CC) $(QEMU_INCLUDES) $(QEMU_CFLAGS) $(QEMU_DGFLAGS) $(CFLAGS) -c -o $@ $<," lt CC $@") > + $(call quiet-command,$(LIBTOOL) --mode=compile --quiet --tag=CC $(CC) $(QEMU_INCLUDES) $(QEMU_CFLAGS) $(QEMU_DGFLAGS) $(CFLAGS) -c -o $@ $<," lt CC $@") > endif > > %.o: %.S -- This message has been scanned for viruses and dangerous content by MailScanner, and is believed to be clean.