Remove the some calls to AnalysisEnvironment.getDerivedArtifact() and add a big fat warning that that

method shouldn't be used anymore.

Ideally, I'd mark it as deprecated, but it's legitimately used by RuleContext. I could just forbid access to AnalysisEnvironment to rule implementations, but, sadly, almost every method on it is called at a few oddball places in implementations of various rules, so we can't do it just yet.

--
MOS_MIGRATED_REVID=101345484

1 files changed, 1 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/SolibSymlinkAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/SolibSymlinkAction.java
index d597465366..7902cbfeb1 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/SolibSymlinkAction.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/SolibSymlinkAction.java
@@ -169,7 +169,7 @@ public final class SolibSymlinkAction extends AbstractAction {
 
     // Ignore libraries that are already represented by the symlinks.
     Root root = configuration.getBinDirectory();
-    Artifact symlink = ruleContext.getAnalysisEnvironment().getDerivedArtifact(symlinkName, root);
+    Artifact symlink = ruleContext.getShareableArtifact(symlinkName, root);
     ruleContext.registerAction(
         new SolibSymlinkAction(ruleContext.getActionOwner(), library, symlink));
     return LinkerInputs.solibLibraryToLink(symlink, library);