diff options
author | Googler <noreply@google.com> | 2016-10-14 23:23:26 +0000 |
---|---|---|
committer | Philipp Wollermann <philwo@google.com> | 2016-10-17 11:19:27 +0000 |
commit | db6f5f26b50e6b0aa3837e7cb3cd595c9ae47ecf (patch) | |
tree | d97891215ed57186cbeccab1e3bd2abedc700050 /src/test/java/com/google | |
parent | b776d6c12e952eb358c1a036cc9d93d8944c4c77 (diff) |
Augment the CppLinkInfo extra action proto with the command line of the linker.
Currently, the proto only includes the linkopt arguments collected from the rules, which are insufficient for reconstructing the linking step in the extra action listener. This CL adds the complete command line needed to re-run the linking step.
--
MOS_MIGRATED_REVID=136211307
Diffstat (limited to 'src/test/java/com/google')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/rules/cpp/CcLibraryConfiguredTargetTest.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/rules/cpp/CcLibraryConfiguredTargetTest.java b/src/test/java/com/google/devtools/build/lib/rules/cpp/CcLibraryConfiguredTargetTest.java index c8de88c0a4..eec083b3dc 100644 --- a/src/test/java/com/google/devtools/build/lib/rules/cpp/CcLibraryConfiguredTargetTest.java +++ b/src/test/java/com/google/devtools/build/lib/rules/cpp/CcLibraryConfiguredTargetTest.java @@ -240,7 +240,7 @@ public class CcLibraryConfiguredTargetTest extends BuildViewTestCase { assertThat(cppLinkInfo.getBuildInfoHeaderArtifactList()) .containsExactlyElementsIn(buildInfoHeaderArtifacts); assertThat(cppLinkInfo.getLinkOptList()) - .containsExactlyElementsIn(action.getLinkCommandLine().getLinkopts()); + .containsExactlyElementsIn(action.getLinkCommandLine().getRawLinkArgv()); } @Test @@ -274,7 +274,7 @@ public class CcLibraryConfiguredTargetTest extends BuildViewTestCase { assertThat(cppLinkInfo.getBuildInfoHeaderArtifactList()) .containsExactlyElementsIn(buildInfoHeaderArtifacts); assertThat(cppLinkInfo.getLinkOptList()) - .containsExactlyElementsIn(action.getLinkCommandLine().getLinkopts()); + .containsExactlyElementsIn(action.getLinkCommandLine().getRawLinkArgv()); } /** |