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.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
LGTM with the minor issue.
Thanks
Galina
zorg/buildbot/builders/LLDBBuilder.py | ||
---|---|---|
96 | 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 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?