| commit | a093405190a69ef5ada4ea57b295a845186461b4 | [log] [tgz] |
|---|---|---|
| author | adonovan <adonovan@google.com> | Tue Dec 10 08:37:45 2019 -0800 |
| committer | Copybara-Service <copybara-worker@google.com> | Tue Dec 10 08:38:44 2019 -0800 |
| tree | 202fd4c7a449400c58181c77cbb96afbbb3181fb | |
| parent | 9588a9e868dd59ef4909ac859c9bfe3b0f0ab524 [diff] |
bazel C++: remove redundant tool methods like ar_executable()
The CcToolchainProvider defines each of these attributes both as a
field and as a method, so x.f is equivalent to x.f(), which breaks an
invariant of the Starlark spec that x.f() should be decomposable into
y=x.f; y().
This change removes the methods.
RELNOTES[INC]:
The following attributes of CcToolchainProvider, which formerly were
accessible both as fields (x.f) and as methods (x.f()), are now only fields:
ld_executable
objcopy_executable
compiler_executable
preprocessor_executable
nm_executable
objdump_executable
ar_executable
strip_executable
Remove the call parens (x.f() -> x.f) if your code uses them as methods.
PiperOrigin-RevId: 284776115
{Fast, Correct} - Choose two
Build and test software of any size, quickly and reliably.
Speed up your builds and tests: Bazel rebuilds only what is necessary. With advanced local and distributed caching, optimized dependency analysis and parallel execution, you get fast and incremental builds.
One tool, multiple languages: Build and test Java, C++, Android, iOS, Go, and a wide variety of other language platforms. Bazel runs on Windows, macOS, and Linux.
Scalable: Bazel helps you scale your organization, codebase, and continuous integration solution. It handles codebases of any size, in multiple repositories or a huge monorepo.
Extensible to your needs: Easily add support for new languages and platforms with Bazel's familiar extension language. Share and re-use language rules written by the growing Bazel community.
Follow our tutorials:
See CONTRIBUTING.md
