Skip to content
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

Bazel 0.24.0 fails on dynamic_runtime_libs coming from toolchain #7795

Closed
or-shachar opened this issue Mar 21, 2019 · 3 comments
Closed

Bazel 0.24.0 fails on dynamic_runtime_libs coming from toolchain #7795

or-shachar opened this issue Mar 21, 2019 · 3 comments
Assignees
Labels
team-Rules-CPP Issues for C++ rules

Comments

@or-shachar
Copy link
Contributor

Relates to #6968

Description of the problem / feature request:

My builds ran fine on bazel 0.23.X.

I tried to use bazel 0.24.0rc7 on one of our repo that is used --crosstool_top=@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:toolchain and --extra_toolchains=@bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/cpp:cc-toolchain-clang-x86_64-default from bazel toolchain commit bazelbuild/bazel-toolchains@31b5dc8

I got this error:

12:07:26  ERROR: /home/builduser/.cache/bazel/_bazel_builduser/d95f300509f1e228190b211abeba17c4/external/bazel_toolchains/configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default/BUILD:57:1: @bazel_toolchains//configs/ubuntu16_04_clang/1.1/bazel_0.20.0/default:cc-compiler-k8: no such attribute 'dynamic_runtime_libs' in 'cc_toolchain' rule

I didn't see anything in documents of this release about deprecating dynamic_runtime_libs. It may be just a missing documentation.

What operating system are you running Bazel on?

Linux

What's the output of bazel info release?

Bazel 0.24.0rc7

Have you found anything relevant by searching the web?

Didn't see anything interesting .. Other than a note for bazel 0.22.0 that dynamic_runtime_libs is optional.

====
Note: Changing the toolchain and crosstool to point to /bazel_toolchains/configs/ubuntu16_04_clang/1.1/bazel_0.22.0/* solved the issue because there the attribute dynamic_runtime_libs does not exist.
Still I think something needs to be documented about it.

@or-shachar
Copy link
Contributor Author

@katre - not sure how to provide steps to reproduce as we're running our builds with Remote Execution service I can share our .bazelrc if that's helpful or you can take some repo like rules_scala and try to play with it's bazelrc

@katre katre added team-Rules-CPP Issues for C++ rules untriaged labels Mar 21, 2019
@hlopko
Copy link
Member

hlopko commented Mar 21, 2019

This was guarded by #6942. It was announced in https://blog.bazel.build/2019/01/28/bazel-0.22.0.html. 0.24 release notes will have the announcement that this flag has been flipped.

@hlopko hlopko closed this as completed Mar 21, 2019
@or-shachar
Copy link
Contributor Author

Great! thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Rules-CPP Issues for C++ rules
Projects
None yet
Development

No branches or pull requests

3 participants