This is an archive of the discontinued LLVM Phabricator instance.

[llvm-zorg]Give default values to BUILDBOT_CLOBBER and BUILDBOT_REVISION
ClosedPublic

Authored by oontvoo on Nov 18 2020, 1:08 PM.

Details

Summary

so you don't have to set them if you don't care about them.

Diff Detail

Event Timeline

oontvoo created this revision.Nov 18 2020, 1:08 PM
oontvoo requested review of this revision.Nov 18 2020, 1:08 PM
vitalybuka accepted this revision.Nov 18 2020, 4:24 PM
vitalybuka added inline comments.
zorg/buildbot/builders/sanitizers/buildbot_functions.sh
23–29

this way is shorter and comment is redundant

This revision is now accepted and ready to land.Nov 18 2020, 4:24 PM
vitalybuka added inline comments.Nov 18 2020, 4:26 PM
zorg/buildbot/builders/sanitizers/buildbot_functions.sh
23–29

off-cause it should be BUILDBOT_REVISION="${BUILDBOT_REVISION:-origin/master}"

oontvoo updated this revision to Diff 306264.Nov 18 2020, 5:03 PM
oontvoo marked 2 inline comments as done.

updated diff

oontvoo added inline comments.Nov 18 2020, 5:03 PM
zorg/buildbot/builders/sanitizers/buildbot_functions.sh
23–29

Thanks!