From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1NXbCW-0001Es-Cn for qemu-devel@nongnu.org; Wed, 20 Jan 2010 09:06:28 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1NXbCR-00017n-R8 for qemu-devel@nongnu.org; Wed, 20 Jan 2010 09:06:27 -0500 Received: from [199.232.76.173] (port=33143 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1NXbCR-00017V-Bf for qemu-devel@nongnu.org; Wed, 20 Jan 2010 09:06:23 -0500 Received: from pig.zood.org ([194.242.112.24]:35205) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1NXbCQ-0003XU-QH for qemu-devel@nongnu.org; Wed, 20 Jan 2010 09:06:23 -0500 Date: Wed, 20 Jan 2010 15:06:21 +0100 From: =?iso-8859-1?Q?Lo=EFc?= Minier Subject: Re: [Qemu-devel] Re: Stop using "which" in ./configure Message-ID: <20100120140621.GA973@pig.zood.org> References: <20100117124528.GA24106@bee.dooz.org> <4B531406.2070904@mail.berlios.de> <20100119101119.GA11358@bee.dooz.org> <20100119114750.GB11613@bee.dooz.org> <20100120113741.GA31679@pig.zood.org> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="J2SCkAp4GZ/dPZZf" Content-Disposition: inline In-Reply-To: <20100120113741.GA31679@pig.zood.org> Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org --J2SCkAp4GZ/dPZZf Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jan 20, 2010, Lo=EFc Minier wrote: > I'm attaching a new patch which changes the tests a bit; I would prefe= r > if someone with access to a Solaris build environment would do this > though. Sorry, there was a typo in the new patch, a test was reversed; updated patch attached. --=20 Lo=EFc Minier --J2SCkAp4GZ/dPZZf Content-Type: text/plain; charset=us-ascii Content-Disposition: attachment; filename="0002-Solaris-test-for-presence-of-commands-with-has.patch" >>From 8b17a25e0f16390a6afed3e030a11ad290f413d7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Minier?= Date: Wed, 20 Jan 2010 12:35:54 +0100 Subject: [PATCH 2/2] Solaris: test for presence of commands with has() --- configure | 8 +++----- 1 files changed, 3 insertions(+), 5 deletions(-) diff --git a/configure b/configure index 90b3c18..4d48341 100755 --- a/configure +++ b/configure @@ -799,21 +799,19 @@ fi # Solaris specific configure tool chain decisions # if test "$solaris" = "yes" ; then - solinst=`path_of $install` - if test -z "$solinst" ; then + if ! has $install; then echo "Solaris install program not found. Use --install=/usr/ucb/install or" echo "install fileutils from www.blastwave.org using pkg-get -i fileutils" echo "to get ginstall which is used by default (which lives in /opt/csw/bin)" exit 1 fi - if test "$solinst" = "/usr/sbin/install" ; then + if "`path_of $install`" = "/usr/sbin/install" ; then echo "Error: Solaris /usr/sbin/install is not an appropriate install program." echo "try ginstall from the GNU fileutils available from www.blastwave.org" echo "using pkg-get -i fileutils, or use --install=/usr/ucb/install" exit 1 fi - sol_ar=`path_of ar` - if test -z "$sol_ar" ; then + if ! has ar; then echo "Error: No path includes ar" if test -f /usr/ccs/bin/ar ; then echo "Add /usr/ccs/bin to your path and rerun configure" -- 1.6.5 --J2SCkAp4GZ/dPZZf--