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

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

 



On Wed, 2019-03-27 at 13:47 -0500, Eric Blake wrote:
[...]
> +# Create mock XDG files/directories to avoid permission problems.
> +# As it points inside $test_dir_, it is automatically cleaned.
> +mock_xdg_()

There doesn't seem to be a consistent pattern deciding when functions
should be named_like_this() instead of being named_like_that_(), so I
guess either one works fine.

> +{
> +  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.

With $XDG_RUNTIME_HOME set correctly,

  Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>

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