Skip to content

Commit

Permalink
Fix small bug in RMG-tests triggering bash script.
Browse files Browse the repository at this point in the history
I hate bash.

this was causing these errors:
./trigger-rmg-tests.sh: line 33: [master: command not found
./trigger-rmg-tests.sh: line 45: [master: command not found

A consequence was the Travis job running RMG-tests thought
everything was a dual RMG-Py/RMG-database job.
They still worked because they used the "master" branch for RMG-Py,
but this way is probably better, and is certainly what was intended.
  • Loading branch information
rwest committed Jun 5, 2021
1 parent 33b14ba commit 93660cb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions trigger-rmg-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ cd $TARGET_DIR

# create a new branch in RMG-tests with the name equal to
# the branch name of the tested RMG-database branch:
if ["$RMG_PY_BRANCH" == "master"]
if [ "$RMG_PY_BRANCH" == "master" ]
then
RMGTESTSBRANCH=rmgdb-$BRANCH
else
Expand All @@ -42,7 +42,7 @@ git checkout $RMGTESTSBRANCH

# create an empty commit with the SHA-ID of the
# tested commit of the RMG-database branch:
if ["$RMG_PY_BRANCH" == "master"]
if [ "$RMG_PY_BRANCH" == "master" ]
then
git commit --allow-empty -m rmgdb-$REV
else
Expand Down

0 comments on commit 93660cb

Please sign in to comment.