Please remove bashism from configure.ac: - if test "x$with_firewalld" == "xyes" ; then + if test "x$with_firewalld" = "xyes" ; then configure gives the following warning with dash: ... checking for GNUTLS... yes checking for gnutls/crypto.h... yes checking for pkcheck... /usr/bin/pkcheck checking whether pkcheck supports uid value... yes ./configure: 63906: test: xno: unexpected operator checking for dtrace... no checking for mntent.h... (cached) yes checking for mount... /bin/mount ...
For issues like this, it's faster to just post your patch directly upstream to libvir-list. But I can take care of it...
commit aca0ae1faa163bbd60ee8df4b93ae870aa820746 Author: Eric Blake <eblake> Date: Wed Nov 19 12:13:14 2014 -0700 maint: use portable shell Reported in https://bugzilla.redhat.com/show_bug.cgi?id=1165827; dash complains: checking for pkcheck... /usr/bin/pkcheck checking whether pkcheck supports uid value... yes ./configure: 63906: test: xno: unexpected operator checking for dtrace... no * configure.ac: Use '=' not '==' in test. Signed-off-by: Eric Blake <eblake>