Skip to content

Commit

Permalink
Use portable sed invocation (#4494)
Browse files Browse the repository at this point in the history
The `\|` operator is not portable to BSD / macOS sed, which makes some
tests appear flaky when run on those platforms. This PR switches the
offending command over to a wordier, but correct invocation.
  • Loading branch information
Baltoli authored Jul 2, 2024
1 parent 144e5db commit cf0152b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion k-distribution/include/kframework/ktest-common.mak
Original file line number Diff line number Diff line change
Expand Up @@ -28,4 +28,4 @@ LLVM_KRUN=${K_BIN}/llvm-krun
# and kdep
KDEP=${K_BIN}/kdep
# command to strip paths from test outputs
REMOVE_PATHS=| sed 's!\('`pwd`'\|'${BUILTIN_DIR}'\|/nix/store/.\+/include/kframework/builtin\)/\(\./\)\{0,2\}!!g'
REMOVE_PATHS=| sed 's!\('`pwd`'\)/\(\./\)\{0,2\}!!g' | sed 's!\('${BUILTIN_DIR}'\)/\(\./\)\{0,2\}!!g' | sed 's!\('/nix/store/..*/include/kframework/builtin'\)/\(\./\)\{0,2\}!!g'

0 comments on commit cf0152b

Please sign in to comment.