commit | aed4160787ba0ec272dad8d31642fee27a16344d | [log] [tgz] |
---|---|---|
author | Googler <noreply@google.com> | Tue Jun 02 12:22:53 2020 -0700 |
committer | Copybara-Service <copybara-worker@google.com> | Tue Jun 02 12:24:52 2020 -0700 |
tree | e5d16a0f2d6c1142dc30dfbee975d222b8c2c6e7 | |
parent | 61a9f57d595492e0cd68758ca63fc24832198554 [diff] |
Clean up artifact retrieval methods in ActionExecutionValue. All callers of getArtifactValue were doing a checkNotNull on the return value, which is exactly what getExistingFileArtifactValue does. Migrate them so that we only need one retrieval method. Loosen the parameter to Artifact and do the instanceof check in one place, so any bugs get an appropriate error message. Additionally, use the recently added isChildOfDeclaredDirectory method to determine which map to search for the artifact, instead of unnecessarily checking both. RELNOTES: None. PiperOrigin-RevId: 314380367
{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