Skip to content

Commit 5b25168

Browse files
author
Kevin Brown
committed
Fix sed issues in the release scripts
Passing in a blank string after `-i` will cause sed to fail, as it tries to interpret it as a file (which does not exist) and bails out.
1 parent 2752563 commit 5b25168

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

release.sh

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,12 +28,14 @@ git pull
2828

2929
echo "Updating Version Identifiers"
3030

31-
sed -E -e "s/\"version\": \"([0-9\.]+)\",/\"version\": \"$ver\",/g" -i "" bower.json select2.jquery.json component.json composer.json package.json
31+
sed -E -e "s/\"version\": \"([0-9\.]+)\",/\"version\": \"$ver\",/g" -i -- bower.json select2.jquery.json component.json composer.json package.json
32+
3233
git add bower.json
3334
git add select2.jquery.json
3435
git add component.json
3536
git add composer.json
3637
git add package.json
38+
3739
git commit -m "modified version identifiers in descriptors for release $ver"
3840
git push
3941

@@ -42,9 +44,10 @@ git checkout "$branch"
4244

4345
echo "Tokenizing..."
4446

45-
find . -name "$js" | xargs -I{} sed -e "$tokens" -i "" {}
46-
find . -name "$css" | xargs -I{} sed -e "$tokens" -i "" {}
47-
sed -e "s/latest/$ver/g" -i "" bower.json
47+
find . -name "$js" | xargs -I{} sed -e "$tokens" -i -- {}
48+
find . -name "$css" | xargs -I{} sed -e "$tokens" -i -- {}
49+
50+
sed -e "s/latest/$ver/g" -i -- bower.json
4851

4952
git add "$js"
5053
git add "$css"

0 commit comments

Comments
 (0)