commit 66da03c3c1e276fa37e1b23647d60d2ef922a866
author ajurkowski <ajurkowski@google.com> Mon Jun 15 15:27:01 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Mon Jun 15 15:27:56 2020 -0700
tree f7745fab46155431f4248f28c90442ac9d31a6ec
parent 5de680aa3b9d99c37954447e6ca7b66d1b551978

Update the Javadoc for `ArtifactRoot#asDerivedRoot`.

Javadoc of ArtifactRoot#asDerivedRoot mentions derived roots registration in
`ArtifactFactory`, which has been removed in the past. Update the comment so it
matches current logic in `ArtifactFactory`.

PiperOrigin-RevId: 316557432

src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java

diff --git a/src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java b/src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java
index 2b65c59..d12e185 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/ArtifactRoot.java
@@ -62,8 +62,8 @@
    * Constructs an ArtifactRoot given the output prefixes. (eg, "bin"), and (eg, "testlogs")
    * relative to the execRoot.
    *
-   * <p>Be careful with this method - all derived roots must be registered with the artifact factory
-   * before the analysis phase.
+   * <p>Be careful with this method - all derived roots must be within the derived artifacts tree,
+   * defined in ArtifactFactory (see {@link ArtifactFactory#isDerivedArtifact(PathFragment)}).
    */
   public static ArtifactRoot asDerivedRoot(Path execRoot, String... prefixes) {
     Path root = execRoot;