diff options
author | rduan <rduan@google.com> | 2017-07-20 22:17:42 +0200 |
---|---|---|
committer | Klaus Aehlig <aehlig@google.com> | 2017-07-21 09:15:27 +0200 |
commit | c003cc2c1d40784cfa24c77388c72ca6bf056592 (patch) | |
tree | b4f2661556f63fff6787df017b113e3a5ef5cf31 /src/main/java/com/google/devtools/build | |
parent | 3476c6d88a1ab81e1618b3dc76ca0eb01e4c3246 (diff) |
Add all generated headers as mandatory inputs for compile actions.
We previously only add those under genfiles root. But J2ObjC and objc proto headers are under the bin root, which we may also fix in the near future.
RELNOTES: None.
PiperOrigin-RevId: 162656166
Diffstat (limited to 'src/main/java/com/google/devtools/build')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCppSemantics.java | 4 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/vfs/PathFragment.java | 8 |
2 files changed, 1 insertions, 11 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCppSemantics.java b/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCppSemantics.java index 51d4c7c9a6..30b9710604 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCppSemantics.java +++ b/src/main/java/com/google/devtools/build/lib/rules/objc/ObjcCppSemantics.java @@ -112,10 +112,8 @@ public class ObjcCppSemantics implements CppSemantics { // TODO(b/62060839): Identify the mechanism used to add generated headers in c++, and recycle // it here. - PathFragment genfilesSegment = - ruleContext.getConfiguration().getGenfilesDirectory().getExecPath().getLastSegment(); for (Artifact header : objcProvider.get(HEADER)) { - if (genfilesSegment.equals(header.getRoot().getExecPath().getLastSegment())) { + if (!header.isSourceArtifact()) { generatedHeaders.add(header); } } diff --git a/src/main/java/com/google/devtools/build/lib/vfs/PathFragment.java b/src/main/java/com/google/devtools/build/lib/vfs/PathFragment.java index 279c8c5dd0..cc9aa199d3 100644 --- a/src/main/java/com/google/devtools/build/lib/vfs/PathFragment.java +++ b/src/main/java/com/google/devtools/build/lib/vfs/PathFragment.java @@ -524,14 +524,6 @@ public abstract class PathFragment implements Comparable<PathFragment>, Serializ } /** - * Returns the last segment in this path, or null if this PathFragment represents the root of the - * filesystem. - */ - public PathFragment getLastSegment() { - return segments.length == 0 ? null : relativeTo(getParentDirectory()); - } - - /** * Returns a new PathFragment formed by appending {@code newName} to the * parent directory. Null is returned iff this method is called on a * PathFragment with zero segments. If {@code newName} designates an absolute path, |