diff options
| author |   wutianwei <wutianwei1@huawei.com> | 2018-02-07 11:48:44 +0800 |
|---|---|---|
| committer | wutianwei <wutianwei1@huawei.com> | 2018-02-07 12:03:02 +0800 |
| commit | ec681c8ad6c012491040bda7b553c32f4d6c4563 (patch) | |
| tree | b5082e41dd58d1edef0c7e51ed947c22bf2ce861 | |
| parent | 0b4524bfcd0e406687e2053cf7ef9913b606f635 (diff) | |
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
| -rw-r--r-- | jjb/global/releng-macros.yml | 4 |
1 files 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..." |