This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Use portable "#!/usr/bin/env bash" shebang for tests.
ClosedPublic

Authored by fcambus on Sep 19 2021, 9:02 AM.

Details

Summary

In build_symbolizer.sh we can safely remove the -eu argument from the shebang (which is an unportable construct), as the scripts sets -e and -u already.

Diff Detail

Event Timeline

fcambus created this revision.Sep 19 2021, 9:02 AM
fcambus requested review of this revision.Sep 19 2021, 9:02 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 19 2021, 9:02 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
xgupta accepted this revision.Sep 24 2021, 6:39 AM
xgupta added a subscriber: xgupta.

LGTM, Thanks!

This revision is now accepted and ready to land.Sep 24 2021, 6:39 AM