Re: [PATCH] tests: Make pci config files writable

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Nov 25, 2013 at 03:22:38PM +0100, Michal Privoznik wrote:
> As of 21685c955 the 'distcheck' is broken. The problem is, by default it
> copies all the necessary files and make them read only. However, pci
> device detach test doesn't work that way. The PCI device configs are
> stored within our tests/ directory and need to be writable (detaching
> and resetting means writing into the config file). Hence, we must make
> those files writable again.
> 
> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> ---
>  tests/Makefile.am | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index e46d5f7..29dbf76 100644
> --- a/tests/Makefile.am
> +++ b/tests/Makefile.am
> @@ -921,3 +921,8 @@ endif ! WITH_CIL
>  
>  CLEANFILES = *.cov *.gcov .libs/*.gcda .libs/*.gcno *.gcno *.gcda *.cmi *.cmx \
>  	object-locking-files.txt
> +
> +# Some tests tend to write into files. Notably, the virpcitest, which detach
> +# and reset a pci device (achieved byt writing into a pci config file).
> +check-local:
> +	chmod -R u+w $(srcdir)/virpcitestdata/

I seem to recall eric saying on IRC that we shouldn't do this, and that
any files the test suite writes to should be located in $(builddir)
not $(srcdir).

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]