On Tue, Oct 22, 2013 at 08:42:12PM +0200, Daniel Exner wrote:
 -dnl For ArchLinux handler.
 -AC_CHECK_PROG(PACMAN,[pacman],[pacman],[no])
 +dnl For FrugalWare handler.
 +AC_CHECK_PROG(PACMAN_G2,[pacman-g2],[pacman-g2],[no])
 +
 +dnl if test "x$pacman_g2" = "xno"; then
 +dnl	dnl For ArchLinux handler.
 +dnl	AC_CHECK_PROG(PACMAN,[pacman],[pacman],[no])
 +dnl fi
 + 
This seems to disable the ArchLinux (ie. pacman) driver.
Is that intentional?  It seems wrong to me.
The rest of the patch is fine.
Rich.
-- 
Richard Jones, Virtualization Group, Red Hat 
http://people.redhat.com/~rjones
libguestfs lets you edit virtual machines.  Supports shell scripting,
bindings from many languages.  
http://libguestfs.org