On 05/25/2012 06:36 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
>
> When adding new config file parameters, the corresponding
> additions to the augeas lens' are constantly forgotten.
> Also there are augeas test cases, these don't catch the
> error, since they too are never updated.
>
> To address this, the augeas test cases need to be auto-generated
> from the example config files.
>
> * build-aux/augeas-gentest.pl: Helper to generate an
> augeas test file, substituting in elements from the
> example config files
> * src/Makefile.am, daemon/Makefile.am: Switch to
> auto-generated augeas test cases
> * daemon/test_libvirtd.aug, daemon/test_libvirtd.aug.in,
> src/locking/test_libvirt_sanlock.aug,
> src/locking/test_libvirt_sanlock.aug.in,
> src/lxc/test_libvirtd_lxc.aug,
> src/lxc/test_libvirtd_lxc.aug.in,
> src/qemu/test_libvirtd_qemu.aug,
> src/qemu/test_libvirtd_qemu.aug.in: Remove example
> config file data, replacing with a ::CONFIG:: placeholder
>
> Signed-off-by: Daniel P. Berrange <berrange@xxxxxxxxxx>
> ---
> +++ b/build-aux/augeas-gentest.pl
> @@ -0,0 +1,51 @@
> +#!/usr/bin/perl
> +
Missing a copyright header.
Nice! ACK, once you fix that, and silence these syntax checks:
TAB_in_indentation
daemon/test_libvirtd.aug.in:15: { "unix_sock_dir" = "/var/run/libvirt" }
maint.mk: indent with space, not TAB, in C, sh, html, py, syms and RNG
schemas
make: *** [sc_TAB_in_indentation] Error 1
prohibit_empty_lines_at_EOF
src/lxc/test_libvirtd_lxc.aug.in
maint.mk: empty line(s) or no newline at EOF
make: *** [sc_prohibit_empty_lines_at_EOF] Error 1
Also, if you rearrange qemu.conf per my comments in 3/4, you might have
to adjust the test file to match here.
--
Eric Blake eblake@xxxxxxxxxx +1-919-301-3266
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