Skip to content

Commit

Permalink
Merge pull request #506 from boegel/2023.06-software.eessi.io_fix-che…
Browse files Browse the repository at this point in the history
…ck-missing-installations

fix check_missing_installations.sh when unmerged PR is used
  • Loading branch information
ocaisa committed Mar 15, 2024
2 parents 06a0413 + a12b9a3 commit 20c5c98
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion check_missing_installations.sh
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ exit_code=${PIPESTATUS[0]}

ok_msg="Command 'eb --missing ...' succeeded, analysing output..."
fail_msg="Command 'eb --missing ...' failed, check log '${eb_missing_out}'"
if [ "$exit_code" -ne 0 ] && [ ! -z $pr_exceptions ]; then
if [ "$exit_code" -ne 0 ] && [ ! -z "$pr_exceptions" ]; then
# We might have failed due to unmerged PRs. Try to make exceptions for --from-pr added in this PR
# to software-layer, and see if then it passes. If so, we can report a more specific fail_msg
# Note that if no --from-pr's were used in this PR, $pr_exceptions will be empty and we might as
Expand Down

0 comments on commit 20c5c98

Please sign in to comment.