- Move common code from buildbot_bootstrap.sh and buildbot_fast.sh to
buildbot_functions.sh.
- Do not capitalize variable names needlessly.
- Use mkdir -p where appropriate instead of "mdkir if directory does not exist".
Paths
| Differential D6710
Refactor sanitizer build scripts. ClosedPublic Authored by earthdok on Dec 17 2014, 12:15 PM.
Details
Summary
buildbot_functions.sh.
Diff Detail Event Timelineearthdok updated this object.
This revision is now accepted and ready to land.Dec 18 2014, 6:11 AM
Revision Contents
Diff 17409 zorg/buildbot/builders/sanitizers/buildbot_bootstrap.sh
zorg/buildbot/builders/sanitizers/buildbot_fast.sh
zorg/buildbot/builders/sanitizers/buildbot_functions.sh
|
I think it would be better if stuff in buildbot_functions.sh did not rely on the STAGE2_MSAN_DIR and similar settings, and instead got them through function arguments. That would make it more reusable.
Ex.
build_stage2_msan "step name" <target directory> <previous stage compiler directory>