[Libguestfs] [PATCH] tests: Provide full path to Unix domain sockets.
Richard W.M. Jones
rjones at redhat.com
Wed Jun 6 18:55:15 UTC 2018
On Wed, Jun 06, 2018 at 01:48:56PM -0500, Eric Blake wrote:
> On 06/06/2018 01:36 PM, Richard W.M. Jones wrote:
> >Maybe a bug in libvirt?
> >https://www.redhat.com/archives/libvir-list/2018-June/msg00490.html
>
> Or maybe intentional. Using absolute paths is better than assuming
> that libvirt will treat relative paths against the same directory as
> you intended; although it would be nicer if libvirt would reject XML
> with a relative path up front rather than look in the wrong
> directory and perhaps succeed by accident on the wrong file.
>
> >---
> > tests/test-cache.sh | 3 ++-
> > tests/test-cow.sh | 3 ++-
> > 2 files changed, 4 insertions(+), 2 deletions(-)
>
> >@@ -69,7 +70,7 @@ cleanup ()
> > trap cleanup INT QUIT TERM EXIT ERR
> > # Open the overlay and perform some operations.
> >-guestfish --format=raw -a 'nbd://?socket=cache.sock' <<'EOF'
> >+guestfish --format=raw -a "nbd://?socket=`pwd`/cache.sock" <<'EOF'
>
> We're using bash, so save a fork, by using $PWD instead of `pwd`.
Thanks, I pushed it using $PWD and a more neutral commit message.
Rich.
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
libguestfs lets you edit virtual machines. Supports shell scripting,
bindings from many languages. http://libguestfs.org
More information about the Libguestfs
mailing list