Re: [PATCH 4/8] tests: don't alter state in $HOME

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

 



On 03/26/2011 08:12 AM, Eric Blake wrote:
Diego reported a bug where virsh tries to initialize a readline
history directory during 'make check' run as root, but fails
because /root was read-only.

It turns out that I could reproduce this as non-root, by using:

mv ~/.virsh{,.bak}
chmod a-w ~
make check -C tests TESTS=int-overflow
chmod u+w ~
mv ~/.virsh{.bak,}

* tests/int-overflow: Don't trigger interactive mode.
Reported by Diego Elio PettenÃ.
---

Original post:
https://www.redhat.com/archives/libvir-list/2011-March/msg01111.html

  tests/int-overflow |    4 ++--
  1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/int-overflow b/tests/int-overflow
index c9f5de9..baf2eef 100755
--- a/tests/int-overflow
+++ b/tests/int-overflow
@@ -14,9 +14,9 @@ fi
  . "$srcdir/test-lib.sh"

  echo "error: failed to get domain '4294967298'">  exp || fail=1
-echo domname 4294967298 | $abs_top_builddir/tools/virsh --quiet \
+$abs_top_builddir/tools/virsh --quiet \
      --connect test://$abs_top_srcdir/examples/xml/test/testnode.xml \
->  /dev/null 2>  err || fail=1
+    'domname 4294967298; quit'>  /dev/null 2>  err || fail=1
  diff -u err exp || fail=1

Why the "; quit" in the command? a quit is implicit anyway.

ACK, with or without the quit - it works either way.

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