Re: [PATCH] D23787: Add builder for clang extra tools Sphinx docs

2016-08-30 Thread Alexander Kornienko via cfe-commits
alexfh added a comment. In https://reviews.llvm.org/D23787#527369, @gribozavr wrote: > 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, https://reviews.llvm.org/D24024 should fix t

Re: [PATCH] D23787: Add builder for clang extra tools Sphinx docs

2016-08-28 Thread Dmitri Gribenko via cfe-commits
gribozavr added a comment. 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'`. Repository: rL LLVM https://reviews.llvm.org/D23787 ___ cfe-commits

Re: [PATCH] D23787: Add builder for clang extra tools Sphinx docs

2016-08-24 Thread Dmitri Gribenko via cfe-commits
gribozavr added a comment. We need to ask Galina Kistanova to reload the configuration. Repository: rL LLVM https://reviews.llvm.org/D23787 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe

Re: [PATCH] D23787: Add builder for clang extra tools Sphinx docs

2016-08-24 Thread Alexander Kornienko via cfe-commits
alexfh added a comment. Do you know whether build slaves should pick this configuration change up automatically? Repository: rL LLVM https://reviews.llvm.org/D23787 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cg

Re: [PATCH] D23787: Add builder for clang extra tools Sphinx docs

2016-08-24 Thread Alexander Kornienko via cfe-commits
This revision was automatically updated to reflect the committed changes. alexfh marked an inline comment as done. Closed by commit rL279624: Add builder for clang extra tools Sphinx docs (authored by alexfh). Changed prior to commit: https://reviews.llvm.org/D23787?vs=68967&id=69110#toc Repos

Re: [PATCH] D23787: Add builder for clang extra tools Sphinx docs

2016-08-24 Thread Dmitri Gribenko via cfe-commits
gribozavr accepted this revision. gribozavr added a comment. This revision is now accepted and ready to land. Thanks, LGTM! https://reviews.llvm.org/D23787 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman

Re: [PATCH] D23787: Add builder for clang extra tools Sphinx docs

2016-08-23 Thread Alexander Kornienko via cfe-commits
alexfh marked an inline comment as done. Comment at: zorg/buildbot/builders/SphinxDocsBuilder.py:46 @@ +45,3 @@ + defaultBranch='trunk', + workdir=clang_srcdir)) + gribozavr wrote: > I think workdir should be `clang_tools_

Re: [PATCH] D23787: Add builder for clang extra tools Sphinx docs

2016-08-23 Thread Alexander Kornienko via cfe-commits
alexfh updated this revision to Diff 68967. alexfh added a comment. - Set proper working directory and build step name for clang extra tools SVN checkout. https://reviews.llvm.org/D23787 Files: buildbot/osuosl/master/config/builders.py zorg/buildbot/builders/SphinxDocsBuilder.py Index: zo

Re: [PATCH] D23787: Add builder for clang extra tools Sphinx docs

2016-08-23 Thread Dmitri Gribenko via cfe-commits
gribozavr added a comment. Just one comment about the workdir. Comment at: zorg/buildbot/builders/SphinxDocsBuilder.py:46 @@ +45,3 @@ + defaultBranch='trunk', + workdir=clang_srcdir)) + I think workdir should be `clang_t

[PATCH] D23787: Add builder for clang extra tools Sphinx docs

2016-08-22 Thread Alexander Kornienko via cfe-commits
alexfh created this revision. alexfh added a reviewer: gribozavr. alexfh added a subscriber: cfe-commits. Cargo-culted from clang docs builder. Should work, but I haven't tried. https://reviews.llvm.org/D23787 Files: buildbot/osuosl/master/config/builders.py zorg/buildbot/builders/SphinxDocs