Re: [PATCH] tests: Avoid writing into $HOME during virsh-snapshot

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

 



On Thu, 2019-03-28 at 10:13 +0100, Andrea Bolognani wrote:
> On Wed, 2019-03-27 at 13:47 -0500, Eric Blake wrote:
> > +  export XDG_CONFIG_HOME="$test_dir_/.config"
> > +  export XDG_CACHE_HOME="$test_dir_/.cache"
> > +  export XDG_RUNTIME_HOME="XDG_CACHE_HOME"
> 
> Aren't you missing the $ in front of XDG_CACHE_HOME here?  The same
> was actually true of the original incarnation of the code, too.

Wait, I actually spotted two more issues with this patch.

First of all, you'll want to add

  /tests/.cache/
  /tests/.config/

to .gitignore; moreover, with this patch applied distcheck will
fail with

  ERROR: files left in build directory after distclean:
  ./tests/.cache/libvirt/virsh/history
  ./tests/.config/libvirt/libvirt.conf
  make[1]: *** [Makefile:2414: distcleancheck] Error 1

so that will have to be addressed as well.

-- 
Andrea Bolognani / Red Hat / Virtualization

--
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]

  Powered by Linux