openstack/devstack
Code Issues Proposed changes

Use proper sed separator for paths

Browse Source 
I941ef5ea90970a0901236afe81c551aaf24ac1d8 added a sed command that
should match and delete path values but used '/' as sed separator. This
leads to error in unstack.sh runs when the path also contains '/':

+./unstack.sh:main:188 sudo sed -i '/directory=/opt/stack/ d' /etc/gitconfig
sed: -e expression #1, char 13: unknown command: `o'

So this patch replace '/' separator with '+'.

Change-Id: I06811c0d9ee7ecddf84ef1c6dd6cff5129dbf4b1
This commit is contained in:
Balazs Gibizer 2022-04-22 12:01:13 +02:00
parent 45f71b10ef
commit 7191c5e7e7
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
unstack.sh
View File

@ -185,4 +185,4 @@ rm -Rf $DEST/async
# Clean any safe.directory items we wrote into the global # Clean any safe.directory items we wrote into the global
# gitconfig. We can identify the relevant ones by checking that they # gitconfig. We can identify the relevant ones by checking that they
# point to somewhere in our $DEST directory. # point to somewhere in our $DEST directory.
sudo sed -i "/directory=${DEST}/ d" /etc/gitconfig sudo sed -i "\+directory = ${DEST}+ d" /etc/gitconfig