-
Notifications
You must be signed in to change notification settings - Fork 691
testing/e2e.sh fails on Mac OS X #217
Comments
|
I'll bet Docker is expecting linux executables, and Mac OS X is building local Darwin, which then is the wrong format. Perhaps we'd need some |
I think that if we do something in the near-term it should be to make this fail-fast saying the platform is unsupported to avoid confusion. This test wasn't intended to cover cross-compilation, which is harder to a variety of reasons (including broad upstream toolchain support). |
That seems like a fine interim solution. It does seem to limit development though, and perhaps we should ensure we have the right issues filed with bazel proper to solve this and other similar requests. I can only imagine that this is an issue that is going to affect the larger docker community who wants to use bazel. |
How hard would it be to build a Linux compiler toolchain inside a Docker image? Since the e2e test already has a dependency on local Docker, we could include a crosstool that knew how to run the compiler from within that image (rather than having to figure out local cross toolchain support for every development host)? |
Good point, we could just have Bazel inside of a Docker image :) The e2e test could use: |
Note that Bazel builds very slowly when running inside docker on OSX because volume IO has to go through Linux virtualization for OSX.. and it's very painful. As a How can we support that in a more robust way? I could do the legwork if I had some guidance from the Bazel/rules_docker team. |
@nictuku You can do this for Go (and probably Java / C++), but it gets challenging for Python / Node.js because they delegate to e.g. I can't say |
For Go, we should add the following to a
It likely requires an update to the version of |
is this still an issue? please reopen if so. |
This quickly fails due to a missing
/bin/true
on line 42 of testing/e2e.sh. Swapping that out for/usr/bin/true
goes much further but still fails:System configuration:
The text was updated successfully, but these errors were encountered: