[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