Skip to content

Commit

Permalink
Fix the exit code check in native test scripts (#1363)
Browse files Browse the repository at this point in the history
We had it double escaped, which meant we ended up with `EXIT_STATUS=\$?`

This doesn't work in bash, it needs to be `EXIT_STATUS=$?`
  • Loading branch information
timyates authored Oct 13, 2023
1 parent 9203a86 commit ce012c4
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ echo "Executing '$folder' $testcopy"
"""
if (nativeTest) {
bashScript += """\
${buildTool == MAVEN ? './mvnw -Pnative test' : './gradlew nativeTest'} || EXIT_STATUS=\\\$?
${buildTool == MAVEN ? './mvnw -Pnative test' : './gradlew nativeTest'} || EXIT_STATUS=\$?
"""
} else {
bashScript += """\
Expand Down

0 comments on commit ce012c4

Please sign in to comment.