Correct header thinning in ObjC crosstool rules so that objc_framework

imports can be discovered.

--
PiperOrigin-RevId: 150882114
MOS_MIGRATED_REVID=150882114

2 files changed, 9 insertions, 4 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/CrosstoolCompilationSupport.java b/src/main/java/com/google/devtools/build/lib/rules/objc/CrosstoolCompilationSupport.java
index 32808abcce..e91de27387 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/objc/CrosstoolCompilationSupport.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/objc/CrosstoolCompilationSupport.java
@@ -274,8 +274,14 @@ public class CrosstoolCompilationSupport extends CompilationSupport {
   }
 
   private IncludeProcessing createIncludeProcessing(
-      Iterable<Artifact> potentialInputs, @Nullable Artifact pchHdr) {
+      Iterable<Artifact> privateHdrs, ObjcProvider objcProvider, @Nullable Artifact pchHdr) {
     if (isHeaderThinningEnabled()) {
+      Iterable<Artifact> potentialInputs =
+          Iterables.concat(
+              privateHdrs,
+              objcProvider.get(HEADER),
+              objcProvider.get(STATIC_FRAMEWORK_FILE),
+              objcProvider.get(DYNAMIC_FRAMEWORK_FILE));
       if (pchHdr != null) {
         potentialInputs = Iterables.concat(potentialInputs, ImmutableList.of(pchHdr));
       }
@@ -305,8 +311,7 @@ public class CrosstoolCompilationSupport extends CompilationSupport {
     ObjcCppSemantics semantics =
         new ObjcCppSemantics(
             objcProvider,
-            createIncludeProcessing(
-                Iterables.concat(privateHdrs, publicHdrs, objcProvider.get(HEADER)), pchHdr),
+            createIncludeProcessing(privateHdrs, objcProvider, pchHdr),
             ruleContext.getFragment(ObjcConfiguration.class),
             isHeaderThinningEnabled(),
             intermediateArtifacts);
diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/HeaderThinning.java b/src/main/java/com/google/devtools/build/lib/rules/objc/HeaderThinning.java
index 8f678af07c..ad528645db 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/objc/HeaderThinning.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/objc/HeaderThinning.java
@@ -167,7 +167,7 @@ public class HeaderThinning implements IncludeProcessing {
     throw new UserExecException(
         String.format(
             "Unable to map header file (%s) found during header scanning of %s."
-                + " This is usually the result of a case mismatch.",
+                + " This is usually the result of a case mismatch",
             missingPath, sourceFile.getExecPathString()));
   }
 }