This is an archive of the discontinued LLVM Phabricator instance.

Add builder for clang extra tools Sphinx docs
ClosedPublic

Authored by alexfh on Aug 22 2016, 4:58 PM.

Details

Summary

Cargo-culted from clang docs builder. Should work, but I haven't tried.

Diff Detail

Repository
rL LLVM

Event Timeline

alexfh updated this revision to Diff 68936.Aug 22 2016, 4:58 PM
alexfh retitled this revision from to Add builder for clang extra tools Sphinx docs.
alexfh updated this object.
alexfh added a reviewer: gribozavr.
alexfh added a subscriber: cfe-commits.
gribozavr edited edge metadata.Aug 23 2016, 2:37 AM

Just one comment about the workdir.

zorg/buildbot/builders/SphinxDocsBuilder.py
46 ↗(On Diff #68936)

I think workdir should be clang_tools_srcdir, which should be set appropriately in the code above.

alexfh updated this revision to Diff 68967.Aug 23 2016, 4:44 AM
alexfh edited edge metadata.
  • Set proper working directory and build step name for clang extra tools SVN checkout.
alexfh marked an inline comment as done.Aug 23 2016, 4:45 AM
alexfh added inline comments.
zorg/buildbot/builders/SphinxDocsBuilder.py
46 ↗(On Diff #68936)

Oops, I knew something was wrong. Fixed this and the build step name.

gribozavr accepted this revision.Aug 24 2016, 6:07 AM
gribozavr edited edge metadata.

Thanks, LGTM!

This revision is now accepted and ready to land.Aug 24 2016, 6:07 AM
This revision was automatically updated to reflect the committed changes.
alexfh marked an inline comment as done.

Do you know whether build slaves should pick this configuration change up automatically?

We need to ask Galina Kistanova to reload the configuration.

Here's the new builder: http://lab.llvm.org:8011/builders/clang-tools-sphinx-docs

Fails with ninja: error: unknown target 'docs-clang-tools-html'.

Here's the new builder: http://lab.llvm.org:8011/builders/clang-tools-sphinx-docs

Fails with ninja: error: unknown target 'docs-clang-tools-html'.

I hope, D24024 should fix the issue.