diff options
author | 2018-07-24 11:38:19 -0700 | |
---|---|---|
committer | 2018-07-24 11:39:34 -0700 | |
commit | bf4123df23b5f93e572cd920f15afba340f92391 (patch) | |
tree | 340562de042f2db4093006c87b6d1f7ebccd441a /src/main/java/com/google/devtools/build/lib/actions/FilesetTraversalParams.java | |
parent | 78930aeca06fa0983eba005b7e1806da46ec4537 (diff) |
Automated rollback of commit f309ad3be36363070e87eef0ee04b12f4956d601.
*** Reason for rollback ***
Fixed duplicate derived inputs bug. Test is in diffbase.
RELNOTES[INC]: If the same artifact is generated by two distinct but identical actions, and a downstream action has both those actions' outputs in its inputs, the artifact will now appear twice in the downstream action's inputs. If this causes problems in Skylark actions, you can use the uniquify=True argument in Args.add_args.
PiperOrigin-RevId: 205863806
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/actions/FilesetTraversalParams.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/actions/FilesetTraversalParams.java | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/actions/FilesetTraversalParams.java b/src/main/java/com/google/devtools/build/lib/actions/FilesetTraversalParams.java index 7740a05b51..75907d4f24 100644 --- a/src/main/java/com/google/devtools/build/lib/actions/FilesetTraversalParams.java +++ b/src/main/java/com/google/devtools/build/lib/actions/FilesetTraversalParams.java @@ -26,6 +26,7 @@ import com.google.devtools.build.lib.util.Fingerprint; import com.google.devtools.build.lib.vfs.PathFragment; import com.google.devtools.build.lib.vfs.Root; import com.google.devtools.build.lib.vfs.RootedPath; +import java.util.Objects; import javax.annotation.Nullable; /** @@ -111,9 +112,7 @@ public interface FilesetTraversalParams { if (o instanceof FilesetTraversalParams.DirectTraversalRoot) { FilesetTraversalParams.DirectTraversalRoot that = (FilesetTraversalParams.DirectTraversalRoot) o; - // Careful! We must compare the artifact owners, which the default {@link Artifact#equals()} - // method does not do. See the comments on {@link ArtifactSkyKey} and http://b/73738481. - return Artifact.equalWithOwner(this.getOutputArtifact(), that.getOutputArtifact()) + return Objects.equals(this.getOutputArtifact(), that.getOutputArtifact()) && (this.getRootPart().equals(that.getRootPart())) && (this.getRelativePart().equals(that.getRelativePart())); } |