-
Notifications
You must be signed in to change notification settings - Fork 434
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
fix bazel build #136
fix bazel build #136
Conversation
Doh, I just cut a release! :) No matter, we'll get this fixed (waiting for dzc to comment on that) and then do a follow-up release. |
This PR looks good but what's odd is that |
I'm running the most recent release. It looks like std.jsonnet.h is not included in the sandbox. $ bazel version
Build label: 0.2.0
Build target: bazel-out/local_linux-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Tue Feb 23 13:08:29 2016 (1456232909)
Build timestamp: 1456232909
Build timestamp as int: 1456232909
$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 15.10
Release: 15.10
Codename: wily
$ uname -a
Linux ubuntu 4.2.0-30-generic #36-Ubuntu SMP Fri Feb 26 00:58:07 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux looking at the previous rule, it seems like it should have worked... |
I also had this on my work laptop:
|
@davidzchen did you try on a fresh clone? |
Interesting. This reproduces on Linux but not on OS X. Let's go ahead and merge these changes. I'll try to isolate a test case and send a bug report to Bazel. LGTM |
We could really use a travis rule for bazel builds :) |
I'll set that up :) |
Build appears to be broken on HEAD
Not sure if this is the best way to fix :)
cc @davidzchen