On 11/16/2010 04:30 AM, Stefan Berger wrote: > On 11/15/2010 09:54 PM, Eric Blake wrote: >> On 11/15/2010 07:18 PM, Stefan Berger wrote: >>>> I've gone ahead and pushed it. >>>> >>> >> diff --git i/src/nwfilter/nwfilter_ebiptables_driver.c >> w/src/nwfilter/nwfilter_ebiptables_driver.c >> index c3a0d3e..7b2a505 100644 >> --- i/src/nwfilter/nwfilter_ebiptables_driver.c >> +++ w/src/nwfilter/nwfilter_ebiptables_driver.c >> @@ -61,7 +61,7 @@ >> #define CMD_DEF_PRE "cmd='" >> #define CMD_DEF_POST "'" >> #define CMD_DEF(X) CMD_DEF_PRE X CMD_DEF_POST >> -#define CMD_EXEC "eval res=\\$(\"${cmd}\")" CMD_SEPARATOR >> +#define CMD_EXEC "eval res=\\$\\(\"${cmd}\"\\)" CMD_SEPARATOR >> #define CMD_STOPONERR(X) \ >> X ? "if [ $? -ne 0 ]; then" \ >> " echo \"Failure to execute command '${cmd}'.\";" \ >> >> > ACK. Works for me and passes the TCK tests.
Pushed now. Thanks for testing; and sorry for the breakage. -- Eric Blake ebl...@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list