[Libvir] make syntax-check fails with bzr checkouts

[Daniel P. Berrange]

On Tue, Apr 29, 2008 at 05:11:59PM +0200, Soren Hansen wrote:
> I seem to be completely unable to get make syntax-checks to function
> properly with my bzr checkout of libvirt[1]. I've attached the output as
> as-is.txt.  I tried adding hacking bzr support into vc-list-files (see
> vc-list-files-bzr.patch), but that didn't quite seem to do the trick, as
> you can see in in vc-list-files-maybe-fixed.log, which is the output
> after I patched vc-list-files.
> 
> I tried CVS, too, and that also fails (see cvs-syntax-check.log).
>
> Is it only meant to work with git?  

It works with CVS - I use it with CVS most of the time.


This....

> --- po-check-1	2008-04-29 17:03:10.452343086 +0200
> +++ po-check-2	2008-04-29 17:03:10.756338852 +0200
> @@ -1,36 +1,36 @@
> -gnulib/lib/gai_strerror.c
> -qemud/qemud.c
> -qemud/remote.c
> -src/conf.c
> -src/console.c
> -src/hash.c
> -src/iptables.c
> -src/libvirt.c
> -src/lxc_conf.c
> -src/lxc_container.c
> -src/lxc_driver.c
> -src/openvz_conf.c
> -src/openvz_driver.c
> -src/proxy_internal.c
> -src/qemu_conf.c
> -src/qemu_driver.c
> -src/remote_internal.c
> -src/sexpr.c
> -src/storage_backend.c
> -src/storage_backend_disk.c
> -src/storage_backend_fs.c
> -src/storage_backend_iscsi.c
> -src/storage_backend_logical.c
> -src/storage_conf.c
> -src/storage_driver.c
> -src/test.c
> -src/util.c
> -src/uuid.c
> -src/virsh.c
> -src/virterror.c
> -src/xen_internal.c
> -src/xend_internal.c
> -src/xm_internal.c
> -src/xml.c
> -src/xmlrpc.c
> -src/xs_internal.c
> +./gnulib/lib/gai_strerror.c
> +./qemud/qemud.c
> +./qemud/remote.c
> +./src/conf.c
> +./src/console.c
> +./src/hash.c
> +./src/iptables.c
> +./src/libvirt.c
> +./src/lxc_conf.c
> +./src/lxc_container.c
> +./src/lxc_driver.c
> +./src/openvz_conf.c
> +./src/openvz_driver.c
> +./src/proxy_internal.c
> +./src/qemu_conf.c
> +./src/qemu_driver.c
> +./src/remote_internal.c
> +./src/sexpr.c
> +./src/storage_backend.c
> +./src/storage_backend_disk.c
> +./src/storage_backend_fs.c
> +./src/storage_backend_iscsi.c
> +./src/storage_backend_logical.c
> +./src/storage_conf.c
> +./src/storage_driver.c
> +./src/test.c
> +./src/util.c
> +./src/uuid.c
> +./src/virsh.c
> +./src/virterror.c
> +./src/xen_internal.c
> +./src/xend_internal.c
> +./src/xm_internal.c
> +./src/xml.c
> +./src/xmlrpc.c
> +./src/xs_internal.c

...says that the vc-list-files script is getting a bogus leading ./ on all
your filenames. So something must be different about Ubuntu versions of
some program that causes this:

    awk -F/ '{                          \
        if (!$1 && $3 !~ /^-/) {        \
          f=FILENAME;                   \
          sub(/CVS\/Entries/, "", f);   \
          print f $2;                   \
        }}'                             \
      $(find ${*-*} -name Entries -print) /dev/null;


to prepend a ./ - it could be the content of CVS/Entries, or perhaps
the find command - if you can figure out which, it ought to be easy
enough to strip off the ./

Regards,
Dan.
-- 
|: Red Hat, Engineering, Boston   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|