From: wutianwei <wutianwei1@huawei.com>
Date: Wed, 7 Feb 2018 03:48:44 +0000 (+0800)
Subject: fix sed command in macros
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=commitdiff_plain;h=ec681c8ad6c012491040bda7b553c32f4d6c4563;p=releng.git

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 <wutianwei1@huawei.com>

Change-Id: I1308e9b1b2111f0033df119e4465df4dda27eac0
---

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..."