This is an archive of the discontinued LLVM Phabricator instance.

Prepare LLDB Windows builder to be able to run tests
ClosedPublic

Authored by zturner on Apr 28 2016, 10:06 AM.

Details

Summary

In a followup patch I want to flip the test flag to True, but I want to do this in preparation first since it is a necessary first step, and if testing doesn't work out and we have to turn them back off, this is still a good change that should stay.

Diff Detail

Repository
rL LLVM

Event Timeline

zturner updated this revision to Diff 55433.Apr 28 2016, 10:06 AM
zturner retitled this revision from to Prepare LLDB Windows builder to be able to run tests.
zturner updated this object.
zturner added a reviewer: gkistanova.
zturner added a subscriber: llvm-commits.
gkistanova accepted this revision.Apr 28 2016, 12:26 PM
gkistanova edited edge metadata.

LGTM with the minor issue.

Thanks

Galina

zorg/buildbot/builders/LLDBBuilder.py
96 ↗(On Diff #55433)

This will use the path separator specific to the buildmaster. Should work for you now, but could introduce issues if we would move buildmaster to, let's say, Windows.

Maybe just hard code the path separator to keep it consistent to the rest of the module?

This revision is now accepted and ready to land.Apr 28 2016, 12:26 PM
This revision was automatically updated to reflect the committed changes.