This is an archive of the discontinued LLVM Phabricator instance.

[Release] Increase test-release.sh verbosity
ClosedPublic

Authored by ro on Feb 3 2023, 1:55 AM.

Details

Summary

test-release.sh is too silent in some cases:

  • Only the build proper is run verbosely, but check-all is not.
  • lit is run without -v, so in case of failures one cannot see what's actually wrong.

This patch fixes both issues, running all ${MAKE} invocations with $Verbose (except for ${MAKE} install where it would only add noise), and running lit with -v.

Tested on x86_64-pc-linux-gnu and arm64-apple-darwin21.6.

Btw., I just noticed that one may want to also echo the ${MAKE} check-all and ${MAKE} check calls for symmetry.

Diff Detail

Event Timeline

ro created this revision.Feb 3 2023, 1:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 3 2023, 1:55 AM
ro requested review of this revision.Feb 3 2023, 1:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 3 2023, 1:55 AM
ro added a comment.Feb 3 2023, 1:56 AM

I forgot: this is certainly something that should go onto the release/16.x branch ASAP.

tstellar accepted this revision.Feb 3 2023, 12:26 PM

LGTM, Thanks, can you open the backport request for release/16.x ?

This revision is now accepted and ready to land.Feb 3 2023, 12:26 PM
This revision was automatically updated to reflect the committed changes.
ro added a comment.Feb 6 2023, 12:35 AM

LGTM, Thanks, can you open the backport request for release/16.x ?

Sure, done as Issue #60553.