Change in vdsm[master]: libvirt configure: drop "--copy" option from "sed" invocation

zhshzhou at linux.vnet.ibm.com zhshzhou at linux.vnet.ibm.com
Thu Aug 8 06:49:04 UTC 2013


Zhou Zheng Sheng has uploaded a new change for review.

Change subject: libvirt configure: drop "--copy" option from "sed" invocation
......................................................................

libvirt configure: drop "--copy" option from "sed" invocation

On Ubuntu, sed has no equivalent to --copy option. The --copy option is
neither in GNU sed [1] nor in Single Unix Specification sed [2].

[1] http://www.gnu.org/software/sed/manual/sed.html
[2] http://pubs.opengroup.org/onlinepubs/9699919799/utilities/sed.html

Change-Id: Ic265df1a5abca0104d044f8c3048cf0643547c70
Signed-off-by: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
---
M lib/vdsm/tool/libvirt_configure.sh.in
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/13/17813/1

diff --git a/lib/vdsm/tool/libvirt_configure.sh.in b/lib/vdsm/tool/libvirt_configure.sh.in
index 963a376..5e94a49 100755
--- a/lib/vdsm/tool/libvirt_configure.sh.in
+++ b/lib/vdsm/tool/libvirt_configure.sh.in
@@ -107,7 +107,7 @@
 # Remove configuration created by vdsm (old "# by vdsm" and the new format)
 # Argument: configuration file that will be inspected
 remove_vdsm_conf() {
-    sed -i --copy -e "/${start_conf_section}/,/${end_conf_section}/d" \
+    sed -i -e "/${start_conf_section}/,/${end_conf_section}/d" \
         -e "/${by_vdsm}/d" "$@"
 }
 


-- 
To view, visit http://gerrit.ovirt.org/17813
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic265df1a5abca0104d044f8c3048cf0643547c70
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list