[Pkg-libvirt-commits] [SCM] Libvirt Debian packaging branch, experimental, updated. debian/0.9.6-2-9-ge975aa5

Guido Günther agx at sigxcpu.org
Wed Nov 2 11:29:01 UTC 2011


The following commit has been merged in the experimental branch:
commit a5d03181e0eaf7ab34ceebc30a6bea7c2cc74e32
Author: Guido Günther <agx at sigxcpu.org>
Date:   Tue Nov 1 19:00:49 2011 +0100

    Drop Autodetect-if-the-remote-nc-command-supports-the-q-o.patch
    
    fixed upstream.

diff --git a/debian/patches/Autodetect-if-the-remote-nc-command-supports-the-q-o.patch b/debian/patches/Autodetect-if-the-remote-nc-command-supports-the-q-o.patch
deleted file mode 100644
index 5579682..0000000
--- a/debian/patches/Autodetect-if-the-remote-nc-command-supports-the-q-o.patch
+++ /dev/null
@@ -1,99 +0,0 @@
-From: =?UTF-8?q?Guido=20G=C3=BCnther?= <agx at sigxcpu.org>
-Date: Fri, 8 Jul 2011 21:07:29 +0200
-Subject: Autodetect if the remote nc command supports the -q option
-
-Based on a patch by Marc Deslauriers <marc.deslauriers at ubuntu.com>
-
-RH: https://bugzilla.redhat.com/show_bug.cgi?id=562176
-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/517478
-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=573172
----
- src/rpc/virnetsocket.c   |   23 ++++++++++++++++++++---
- tests/virnetsockettest.c |   11 ++++++-----
- 2 files changed, 26 insertions(+), 8 deletions(-)
-
-diff --git a/src/rpc/virnetsocket.c b/src/rpc/virnetsocket.c
-index 88dc2a4..f08846f 100644
---- a/src/rpc/virnetsocket.c
-+++ b/src/rpc/virnetsocket.c
-@@ -632,9 +632,26 @@ int virNetSocketNewConnectSSH(const char *nodename,
-                              "-e", "none", NULL);
-     if (noVerify)
-         virCommandAddArgList(cmd, "-o", "StrictHostKeyChecking=no", NULL);
--    virCommandAddArgList(cmd, nodename,
--                         netcat ? netcat : "nc",
--                         "-U", path, NULL);
-+
-+    virCommandAddArgList(cmd, nodename, "sh", "-c", NULL);
-+    /*
-+     * This ugly thing is a shell script to detect availability of
-+     * the -q option for 'nc': debian and suse based distros need this
-+     * flag to ensure the remote nc will exit on EOF, so it will go away
-+     * when we close the connection tunnel. If it doesn't go away, subsequent
-+     * connection attempts will hang.
-+     *
-+     * Fedora's 'nc' doesn't have this option, and defaults to the desired
-+     * behavior.
-+     */
-+    virCommandAddArgFormat(cmd,
-+         "'if %s -q 2>&1 | grep \"requires an argument\" >/dev/null 2>&1; then"
-+         "     ARG=-q0;"
-+         "fi;"
-+         "%s $ARG -U %s'",
-+         netcat ? netcat : "nc",
-+         netcat ? netcat : "nc",
-+         path);
- 
-     return virNetSocketNewConnectCommand(cmd, retsock);
- }
-diff --git a/tests/virnetsockettest.c b/tests/virnetsockettest.c
-index fae15a3..b3a2705 100644
---- a/tests/virnetsockettest.c
-+++ b/tests/virnetsockettest.c
-@@ -496,7 +496,7 @@ mymain(void)
-     struct testSSHData sshData1 = {
-         .nodename = "somehost",
-         .path = "/tmp/socket",
--        .expectOut = "somehost nc -U /tmp/socket\n",
-+        .expectOut = "somehost sh -c 'if nc -q 2>&1 | grep \"requires an argument\" >/dev/null 2>&1; then     ARG=-q0;fi;nc $ARG -U /tmp/socket'\n",
-     };
-     if (virtTestRun("SSH test 1", 1, testSocketSSH, &sshData1) < 0)
-         ret = -1;
-@@ -509,7 +509,7 @@ mymain(void)
-         .noTTY = true,
-         .noVerify = false,
-         .path = "/tmp/socket",
--        .expectOut = "-p 9000 -l fred -T -o BatchMode=yes -e none somehost netcat -U /tmp/socket\n",
-+        .expectOut = "-p 9000 -l fred -T -o BatchMode=yes -e none somehost sh -c 'if netcat -q 2>&1 | grep \"requires an argument\" >/dev/null 2>&1; then     ARG=-q0;fi;netcat $ARG -U /tmp/socket'\n",
-     };
-     if (virtTestRun("SSH test 2", 1, testSocketSSH, &sshData2) < 0)
-         ret = -1;
-@@ -522,7 +522,7 @@ mymain(void)
-         .noTTY = false,
-         .noVerify = true,
-         .path = "/tmp/socket",
--        .expectOut = "-p 9000 -l fred -o StrictHostKeyChecking=no somehost netcat -U /tmp/socket\n",
-+        .expectOut = "-p 9000 -l fred -o StrictHostKeyChecking=no somehost sh -c 'if netcat -q 2>&1 | grep \"requires an argument\" >/dev/null 2>&1; then     ARG=-q0;fi;netcat $ARG -U /tmp/socket'\n",
-     };
-     if (virtTestRun("SSH test 3", 1, testSocketSSH, &sshData3) < 0)
-         ret = -1;
-@@ -538,7 +538,8 @@ mymain(void)
-     struct testSSHData sshData5 = {
-         .nodename = "crashyhost",
-         .path = "/tmp/socket",
--        .expectOut = "crashyhost nc -U /tmp/socket\n",
-+        .expectOut = "crashyhost sh -c 'if nc -q 2>&1 | grep \"requires an argument\" >/dev/null 2>&1; then     ARG=-q0;fi;nc $ARG -U /tmp/socket'\n",
-+
-         .dieEarly = true,
-     };
-     if (virtTestRun("SSH test 5", 1, testSocketSSH, &sshData5) < 0)
-@@ -549,7 +550,7 @@ mymain(void)
-         .path = "/tmp/socket",
-         .keyfile = "/root/.ssh/example_key",
-         .noVerify = true,
--        .expectOut = "-i /root/.ssh/example_key -o StrictHostKeyChecking=no example.com nc -U /tmp/socket\n",
-+        .expectOut = "-i /root/.ssh/example_key -o StrictHostKeyChecking=no example.com sh -c 'if nc -q 2>&1 | grep \"requires an argument\" >/dev/null 2>&1; then     ARG=-q0;fi;nc $ARG -U /tmp/socket'\n",
-     };
-     if (virtTestRun("SSH test 6", 1, testSocketSSH, &sshData6) < 0)
-         ret = -1;
--- 

-- 
Libvirt Debian packaging



More information about the Pkg-libvirt-commits mailing list