diff options
author | 2018-03-06 01:58:35 -0800 | |
---|---|---|
committer | 2018-03-06 02:00:07 -0800 | |
commit | 5494ce4d63660f0c958a56c9cf6a5a8836967a97 (patch) | |
tree | bbcabe122d516b4ea46611e1b51f250fcc34553b /src/test/java/com/google/devtools/build/lib/rules/objc/ObjcLibraryTest.java | |
parent | 903b47985b329575dc369ed81dc7194db83a8beb (diff) |
Remove cc_inc_library from bazel
It shouldn't be open sourced at all, since it solves a very specific internal
use case, and for external use cc_library with its includes, include_prefix, and
strip_include_prefix attributes can be used to perfectly replace cc_inc_library.
RELNOTES: Removed cc_inc_library, please use cc_library instead
Use includes, include_prefix, and strip_include_prefix attributes of cc_library
to replace the behavior of cc_inc_library with cc_library.
PiperOrigin-RevId: 187991313
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/rules/objc/ObjcLibraryTest.java')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/rules/objc/ObjcLibraryTest.java | 22 |
1 files changed, 8 insertions, 14 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/rules/objc/ObjcLibraryTest.java b/src/test/java/com/google/devtools/build/lib/rules/objc/ObjcLibraryTest.java index 0a83e02975..8bb93d66f8 100644 --- a/src/test/java/com/google/devtools/build/lib/rules/objc/ObjcLibraryTest.java +++ b/src/test/java/com/google/devtools/build/lib/rules/objc/ObjcLibraryTest.java @@ -48,7 +48,6 @@ import com.google.devtools.build.lib.analysis.actions.ParameterFileWriteAction; import com.google.devtools.build.lib.analysis.config.BuildConfiguration; import com.google.devtools.build.lib.analysis.config.CompilationMode; import com.google.devtools.build.lib.analysis.util.ScratchAttributeWriter; -import com.google.devtools.build.lib.cmdline.RepositoryName; import com.google.devtools.build.lib.packages.NoSuchTargetException; import com.google.devtools.build.lib.packages.util.MockObjcSupport; import com.google.devtools.build.lib.rules.apple.ApplePlatform; @@ -1411,15 +1410,14 @@ public class ObjcLibraryTest extends ObjcRuleTestCase { "cc_library(", " name = 'cc_lib_impl',", " srcs = [", - " 'v1/a.c',", - " 'v1/a.h',", + " 'a.c',", + " 'a.h',", " ],", ")", "", - "cc_inc_library(", + "cc_library(", " name = 'cc_lib',", - " hdrs = ['v1/a.h'],", - " prefix = 'v1',", + " hdrs = ['a.h'],", " deps = [':cc_lib_impl'],", ")"); createLibraryTargetWriter("//objc2:lib") @@ -1573,10 +1571,10 @@ public class ObjcLibraryTest extends ObjcRuleTestCase { " ],", ")", "", - "cc_inc_library(", + "cc_library(", " name = 'cc_lib',", " hdrs = ['v1/a.h'],", - " prefix = 'v1',", + " strip_include_prefix = 'v1',", " deps = [':cc_lib_impl'],", ")"); @@ -1593,12 +1591,8 @@ public class ObjcLibraryTest extends ObjcRuleTestCase { String compileActionArgs = Joiner.on("") .join(compileAction.getArguments()) .replace(" ", ""); - String includeDir = - getAppleCrosstoolConfiguration() - .getIncludeDirectory(RepositoryName.MAIN) - .getExecPathString() - + "/third_party/cc_lib/_/cc_lib"; - assertThat(compileActionArgs).contains("-I" + includeDir); + assertThat(compileActionArgs) + .matches(".*-iquote.*/third_party/cc_lib/_virtual_includes/cc_lib.*"); } @Test |