From ec681c8ad6c012491040bda7b553c32f4d6c4563 Mon Sep 17 00:00:00 2001 From: wutianwei Date: Wed, 7 Feb 2018 11:48:44 +0800 Subject: fix sed command in macros The char "/" was included in $msg. It will have issue "sed: -e expression #1, char 7: unknown option to `s'" We can substitute '/' with '#' in sed command to solve this issue Signed-off-by: wutianwei Change-Id: I1308e9b1b2111f0033df119e4465df4dda27eac0 --- jjb/global/releng-macros.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/jjb/global/releng-macros.yml b/jjb/global/releng-macros.yml index bd9f97d3b..a7d947f81 100644 --- a/jjb/global/releng-macros.yml +++ b/jjb/global/releng-macros.yml @@ -320,7 +320,7 @@ if [[ -s violation.log ]]; then echo "Reporting lint result..." msg="Found syntax error and/or coding style violation(s) in the files modified by your patchset." - sed -i -e '1s/^//$msg\n\n/' violation.log + sed -i -e "1s#^#${msg}\n\n#" violation.log cmd="gerrit review -p $GERRIT_PROJECT -m \"$(cat violation.log)\" $GERRIT_PATCHSET_REVISION --notify NONE" ssh -p 29418 gerrit.opnfv.org "$cmd" @@ -348,7 +348,7 @@ builders: - shell: | #!/bin/bash - # Install python package + # Install python package sudo pip install "flake8==2.6.2" echo "Checking python code..." -- cgit 1.2.3-korg