Re: [PATCH] nwfilter: use /bin/sh rather than requiring bash

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

 



On 11/13/2010 05:04 PM, Stefan Berger wrote:
>> @@ -427,7 +430,7 @@ static int iptablesLinkIPTablesBaseChain(const
>> char *iptables_cmd,
>>                         "    " CMD_DEF("%s -I %s %d -j %s") CMD_SEPARATOR
>>                         "    " CMD_EXEC
>>                         "    %s"
>> -                      "    let r=r+1\n"
>> +                      "    r=$(( $r + 1 ))\n"
> 
> /bin/sh on my system points to bash. What else is it allowed to point
> to?

Debian-based systems set it to dash, which doesn't understand let; and
we do have examples of people in this configuration.  Also, someone
going for a minimal-weight hypervisor might want to use busybox sh,
another POSIX shell that lacks let; but I'm not sure of anyone using
both busybox and kvm-qemu at the same time.

> 
> Since you are the expert with shells and I trust that the TCK tests now
> pass with all possible /bin/sh's, you get my ACK.

Hmm - I guess I'd better test with dash and the TCK before actually
pushing this.

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

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]