diff options
author | plf <plf@google.com> | 2018-02-27 04:16:24 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-02-27 04:18:08 -0800 |
commit | 4c763e8ff61884952953b7bca8793796f282cdc0 (patch) | |
tree | 7b2ae474b66b0e87eeffa21672988d0bc15d0590 /src/main/java/com/google/devtools/build/lib/skyframe/packages | |
parent | 78fa94a6183d417216638144a5e6d0598f1b1afd (diff) |
Java: Stops adding C++ providers directly.
unknown commit fixes the only related broken target showing up in a TGP after
this change. It seems that this provider was only ever used through
java_wrap_cc with java_library dependencies which themselves had cc_library
dependencies, i.e. java_wrap_cc -> java_library -> cc_library. This dependency
is not needed, the cc_library can be added directly to java_wrap_cc to get the
same behavior and this is the way it should be done instead of relying on
transitive dependencies.
The reason for making this change right now is that with the Skylark C++ API, I
will be adding attr("deps").mandatoryProviders(CcCompilationInfo,
CcLinkParamsInfo) to C++ rules so that they can take any Skylark rule and not
having to explicitly hard-code the name of those Skylark rules in Blaze.
However, Java meets this criteria and the change would allow C++ rules to
depend on Java. This is not something that was supported before and is not
something we plan to support.
RELNOTES:none
PiperOrigin-RevId: 187158176
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/skyframe/packages')
0 files changed, 0 insertions, 0 deletions