-
Notifications
You must be signed in to change notification settings - Fork 138
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add kythe.io job #4
Comments
http://www.kythe.io/contributing for how to run kythe Runs on Linux and OS X problems:
@schroederc fyi |
tools/modules/configure.sh is pulling llvm and clang sources... but also requires binaries, why @shahms? |
We depend on the clang AST for the C++ indexer (and hence need the sources) and also need a compiler. @zrlk may know more. |
To which binaries are you referring? We use Clang to build our code and require some version of it to be available. |
I need to |
We build LLVM and Clang libraries for indexing near HEAD, but we rely on a versioned copy of the compiler--that we expect most developers to already have installed--to do the work. |
Another issue with kythe: it's more than the biggest job we have on ci (tensorflow) but the refactor I am doing will also take care of that (as well as handling the configure.sh issue). It will take some weeks though. |
Ok so the work on ci.bazel.io is done, we just need the modification to kythe for not needing to run bazel inside an external script then we can start adding the needed depedencies on the slave and add kythe. |
Closing, as I'm currently not planning on adding more projects to our CI until we're in a more stable state. |
@philwo, would it be possible to reopen this and add Kythe to the Bazel test suite? It might help prevent issues like bazelbuild/bazel#8226 and would help provide coverage over some of the more exotic Bazel APIs |
Yep, @lberki also pinged me about this. It would be best to file a new issue with our "Adding a project to Bazel CI" template. If someone from the Kythe team could do it, this would be perfect. |
No description provided.
The text was updated successfully, but these errors were encountered: