-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Dev build failes: no matching toolchains found for types @bazel_tools//tools/cpp:toolchain_type #1766
Comments
I installed bazel 0.26.1 with the shell installer and updated the rules to 0.18.5. I also see that issue:
|
It looks like this broke in Bazel 0.26.0. I can reproduce this with 0.27.1 as well. It appears to be related to this: bazel-contrib/rules_go#2090 & bazel-contrib/rules_go#2089 I'm going to give running |
@munnerz did master of go rules work? |
Hi Unfortunately I cannot revert to a working version. Also this looks like a very unstable build tool if they brake in such unfixable manners. What is the reason for using it over simple shell and go build tools? It's more typing but for people like me who do not program bazel or whatever that language is called It is simple to chime in and fix it in like 15minutes. With Bazel I unfortunately cannot fix the build quickly and then contribute that. I first would need to learn some exotic language to help out. |
@chrislovecnm as you observed, when using master of rules_go I encountered similar toolchain issues with rules_docker. It seems the steps documented here: bazel-contrib/rules_go#2089 (comment) are required to make crossbuilding work with Bazel 0.26+, and it seems like it could be a little while before all that lands 😞 @Toasterson 0.25.3 is confirmed working for me, including crossbuilding (which is what you are doing here). bazelisk makes it easy to switch Bazel version without having to mess with your system's package manager. I think we may be best to update the repository to include a |
@munnerz Ok thanks that should allow me to work on some changes if I get the contract at $WORK or if i feel like doing it anyway. However I still would like to note that I am not too happy with having to maintain multiple versions of a tool that at the end of the day just executes go and kubernetes commands. I know I am a purist in that regard and not everyone is of that opinion but keeping it simple has its benefits when it comes to attracting contributions. /my2cents EDIT: Having a Contributions Guide without Bazel would be nice |
This is fixed by bazel-contrib/rules_go#2118 You can use master or wait for 0.19. |
I've opened #1855 which bumps us to use that patch, thanks @chrislovecnm 😄 @Toasterson we use Bazel to manage dependencies on Docker images used during e2e tests, as well as many other aspects of the build process. This sort of failure/regression is frustrating, but usually does not last too long and can be worked around. Whilst for some cases it can be easier to write everything ourselves with Bash/Make etc., it becomes gradually more difficult for us to manage dependency versions as we depend upon more and more areas. In the past we took this approach, and found ourselves writing a very lackluster system for pinning versions that everything uses, as well as then supporting those tools across multiple OS/arch. After #1855 merges, you should be able to build all aspects of the project again using any version of Bazel. Thanks for the feedback though, and definitely +1 on improving our contributing documentation. I really appreciate your interest in contributing 💯😄 |
Describe the bug:
A clear and concise description of what the bug is.
Unable to make a local build. Fails with error:
Expected behavior:
A concise description of what you expected to happen.
Successful dev build.
Steps to reproduce the bug:
Steps to reproduce the bug should be clear and easily reproducible to help people
gain an understanding of the problem.
Follow the official instruction
Anything else we need to know?:
Environment details::
bazelbuild/tap/bazel 0.26.1
/kind bug
The text was updated successfully, but these errors were encountered: