[libvirt] [PATCH] configure: Remove bashism and replace 'test ==' with 'test ='

Eric Blake eblake at redhat.com
Fri Nov 12 21:14:18 UTC 2010


On 11/12/2010 01:59 PM, Eric Blake wrote:
> On 11/12/2010 01:50 PM, Matthias Bolte wrote:
>> This also fixes configure problems on FreeBSD, as test doesn't
>> understand '==' there.
>> ---
>>  configure.ac |    6 +++---
>>  1 files changed, 3 insertions(+), 3 deletions(-)
> 
> I'm surprised 'make syntax-check' isn't catching these.  Maybe I can
> whip that one into shape quickly...
> 
>>
>> diff --git a/configure.ac b/configure.ac
>> index f691c27..66237e1 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -269,9 +269,9 @@ fi
>>  
>>  AC_MSG_CHECKING([where to write libvirtd PID file])
>>  AC_ARG_WITH([remote-pid-file], [AC_HELP_STRING([--with-remote-pid-file=@<:@pidfile|none@:>@], [PID file for libvirtd])])
>> -if test "x$with_remote_pid_file" == "x" ; then
>> +if test "x$with_remote_pid_file" = "x" ; then
> 
> Meanwhile, ACK.

I just added a gnulib 'make syntax-check' rule, which caught one
additional instance:

nwfilter_ebiptables_driver.c:653

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20101112/dcc101ef/attachment-0001.sig>


More information about the libvir-list mailing list