This is an archive of the discontinued LLVM Phabricator instance.

Enable tests on the Windows LLDB buildbot
ClosedPublic

Authored by zturner on May 6 2016, 8:32 AM.

Details

Reviewers
gkistanova
Summary

Is this all that is necessary to get this working? ninja check-lldb doesn't use lit but a different testing system, so I don't know if special logic is needed to get buildbot to understand the output of check-lldb, or if it will just work.

Also, how do I add myself to the email list so I get notified whenever this buildbot fails, even if it didn't fail because of me?

Diff Detail

Event Timeline

zturner updated this revision to Diff 56419.May 6 2016, 8:32 AM
zturner retitled this revision from to Enable tests on the Windows LLDB buildbot.
zturner updated this object.
zturner added a reviewer: gkistanova.
zturner added a subscriber: llvm-commits.
gkistanova accepted this revision.May 9 2016, 10:41 AM
gkistanova edited edge metadata.

LGTM.

Is this all that is necessary to get this working?

This should be enough to get it working in general. Depending on the degree of understanding you are after, you might need to implement a custom parser for the tests output, but having just this looks like a good start. ninja check-lldb doesn't use lit but a different testing system, so I don't know if special logic is needed to get buildbot to understand the output of check-lldb, or if it will just work.

Also, how do I add myself to the email list so I get notified whenever this buildbot fails, even if it didn't fail because of me?

You can add MailNotifier to status.py for this.

Thanks

Galina

This revision is now accepted and ready to land.May 9 2016, 10:41 AM

I had to revert this for now as I work through a couple of issues. Is
there any way to get the FULL PATH of the build directory on the slave?
Right now I'm specifying -DLLDB_TEST_COMPILER="bin\clang.exe", but I need
this to be the full path, not a relative path.

Sure. You can do something like

  1. Determine the build directory. f.addStep(SetProperty(name="get_builddir", command=["pwd"], property="builddir", description="set build dir", workdir="."))

and then use it with WithProperties similar to this:

-DLLDB_TEST_COMPILER=WithProperties('%(builddir)s/<whetver-path-from-there>/bin/clang.exe')

There is a usage example in getLLDBBuildFactory in LLDBBuilder.py.

By the way, with r269342 I ttried to disable tests and enable checking out
and building LLD. But for some reason, neither change seems to have been
picked up by the buildbot yet. When I enabled tests, it seemed to take
effect almost immediately, so I'm wondering if I'm missing something or if
it will take some time?

The changes pick up after master updates.
I will update the master tonight.

Thanks

Galina

Eugene.Zelenko added a subscriber: Eugene.Zelenko.

Committed in r269306.