diff options
| author |  hlopko <hlopko@google.com> | 2018-08-07 03:12:29 -0700 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2018-08-07 03:13:51 -0700 |
| commit | bee5fd63681c8ee5e8b4ae0019c0ab23ef3876af (patch) | |
| tree | cf512c982d330ffebf6a6c43ba1b4285ad26d753 | |
| parent | 808ec9ff9b5cec14f23a4ba106bc5249cacc8c54 (diff) | |
Remove is_cc_test_link_action and is_not_cc_test_link_action build variables
All users have been migrated, so it's not used anymore.
RELNOTES: None.
PiperOrigin-RevId: 207688604
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java | 4 | ||||
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariables.java | 6 |
2 files changed, 2 insertions, 8 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java index 1bb49fecaa..26f651bc77 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppActionConfigs.java @@ -597,13 +597,13 @@ public class CppActionConfigs { " flag_group {", ifTrue( supportsEmbeddedRuntimes, - " expand_if_all_available: 'is_cc_test_link_action'", + " expand_if_true: 'is_cc_test'", // TODO(b/27153401): This should probably be @loader_path on osx. " flag: ", " '-Wl,-rpath,$EXEC_ORIGIN/%{runtime_library_search_directories}'", " }", " flag_group {", - " expand_if_all_available: 'is_not_cc_test_link_action'"), + " expand_if_false: 'is_cc_test'"), ifLinux( platform, " flag: '-Wl,-rpath,$ORIGIN/%{runtime_library_search_directories}'"), diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariables.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariables.java index a26b8b76cc..7278aa847d 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariables.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/LinkBuildVariables.java @@ -75,10 +75,6 @@ public enum LinkBuildVariables { FORCE_PIC("force_pic"), /** Presence of this variable indicates that the debug symbols should be stripped. */ STRIP_DEBUG_SYMBOLS("strip_debug_symbols"), - @Deprecated - IS_CC_TEST_LINK_ACTION("is_cc_test_link_action"), - @Deprecated - IS_NOT_CC_TEST_LINK_ACTION("is_not_cc_test_link_action"), /** Truthy when current action is a cc_test linking action, falsey otherwise. */ IS_CC_TEST("is_cc_test"), /** @@ -149,10 +145,8 @@ public enum LinkBuildVariables { if (useTestOnlyFlags) { buildVariables.addIntegerVariable(IS_CC_TEST.getVariableName(), 1); - buildVariables.addStringVariable(IS_CC_TEST_LINK_ACTION.getVariableName(), ""); } else { buildVariables.addIntegerVariable(IS_CC_TEST.getVariableName(), 0); - buildVariables.addStringVariable(IS_NOT_CC_TEST_LINK_ACTION.getVariableName(), ""); } if (runtimeLibrarySearchDirectories != null) { |