-
Notifications
You must be signed in to change notification settings - Fork 17
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
use new native-image options to use bazel's cc #10
Comments
I had a quick look at I get this when I print the result of calling
If I then pass in /usr/bin/gcc and /usr/bin/ld in the path (regardless of using the new option provided inoracle/graal#1557), I get
Which is not inside of As a workaround, the rules execute fine on my system when I add
on the Any ideas of where I could look next? |
It looks like gcc itself is calling as. So I guess we go one layer down the rabbit hole. I can see a few possibilities:
|
Possibly relevant: https://stackoverflow.com/questions/14454404/how-to-choose-the-assembler-gcc-uses#14455196 In 2013 the path to as was a compile time option. If this is still the case, this means we should probably fix it in bazel by getting |
https://gist.github.com/xianyi/2957847 since we are calling clang on os x for gcc i think this approach could work too to redirect the as call back onto clang. |
This page: https://docs.bazel.build/versions/master/cc-toolchain-config-reference.html#assembler-actions References assembler actions, so it seems that bazel must know about an assembler. |
I was poking at this a bit today and I'm not sure I'm anywhere closer than above. I used Unfortunately I started a PR but it's not actually working yet. Also includes downloading from the new URL structure that includes Java version. https://github.com/snoble/rules_graal/pull/1/files |
see: oracle/graal#1557
It may be possible to make this ruleset hermetic with those options. If not, it would be really nice to file the issues we need to make these rules have passing test on sandboxed linux.
cc @ianoc-stripe
The text was updated successfully, but these errors were encountered: