Merge pull request #616 from xymus/c-compat-modulemaps

xymus · web-flow · commit 8888752a1f07 · 2025-06-30T12:16:23.000-07:00
Don't require objc in modulemaps generated for Swift compatibility headers
diff --git a/Sources/SWBTaskConstruction/TaskProducers/OtherTaskProducers/ModuleMapTaskProducer.swift b/Sources/SWBTaskConstruction/TaskProducers/OtherTaskProducers/ModuleMapTaskProducer.swift
@@ -404,7 +404,6 @@ final class ModuleMapTaskProducer: PhasedTaskProducer, TaskProducer {
                 outputStream <<< "module \(try moduleName.asModuleIdentifierString()).Swift {\n"
             }
             outputStream <<< "  header \"\(interfaceHeaderName.asCStringLiteralContent)\"\n"
-            outputStream <<< "  requires objc\n"
             outputStream <<< "}\n"
 
             return outputStream.bytes
diff --git a/Tests/SWBTaskConstructionTests/ModuleMapTaskConstructionTests.swift b/Tests/SWBTaskConstructionTests/ModuleMapTaskConstructionTests.swift
@@ -584,7 +584,6 @@ fileprivate struct ModuleMapTaskConstructionTests: CoreBasedTests {
                     #expect(contents == (OutputByteStream()
                                          <<< "framework module SwiftOnly {\n"
                                          <<< "  header \"SwiftOnly-Swift.h\"\n"
-                                         <<< "  requires objc\n"
                                          <<< "}\n").bytes)
                 }
 
@@ -629,7 +628,6 @@ fileprivate struct ModuleMapTaskConstructionTests: CoreBasedTests {
                                          <<< "\n"
                                          <<< "module ObjCCompatibilityHeader.Swift {\n"
                                          <<< "  header \"ObjCCompatibilityHeader-Swift.h\"\n"
-                                         <<< "  requires objc\n"
                                          <<< "}\n").bytes)
                 }
 
@@ -978,7 +976,6 @@ fileprivate struct ModuleMapTaskConstructionTests: CoreBasedTests {
                                                      <<< "\n"
                                                      <<< "module \(targetName).Swift {\n"
                                                      <<< "  header \"\(targetName)-Swift.h\"\n"
-                                                     <<< "  requires objc\n"
                                                      <<< "}\n").bytes)
                             }
 
diff --git a/Tests/SWBTaskConstructionTests/SwiftTaskConstructionTests.swift b/Tests/SWBTaskConstructionTests/SwiftTaskConstructionTests.swift
@@ -614,7 +614,6 @@ fileprivate struct SwiftTaskConstructionTests: CoreBasedTests {
                 #expect(contents == (OutputByteStream()
                                      <<< "framework module CoreFoo {\n"
                                      <<< "  header \"CoreFoo-Swift.h\"\n"
-                                     <<< "  requires objc\n"
                                      <<< "}\n").bytes)
             }
         }
@@ -1094,7 +1093,6 @@ fileprivate struct SwiftTaskConstructionTests: CoreBasedTests {
                     stream <<< "\n"
                     stream <<< "module CoreFoo.Swift {\n"
                     stream <<< "  header \"CoreFoo-Swift.h\"\n"
-                    stream <<< "  requires objc\n"
                     stream <<< "}\n"
                     #expect(contents == stream.bytes)
 

Original file line number	Diff line number	Diff line change
`@@ -404,7 +404,6 @@ final class ModuleMapTaskProducer: PhasedTaskProducer, TaskProducer {`
`404`	`404`	`outputStream <<< "module \(try moduleName.asModuleIdentifierString()).Swift {\n"`
`405`	`405`	`}`
`406`	`406`	`outputStream <<< " header \"\(interfaceHeaderName.asCStringLiteralContent)\"\n"`
`407`		`- outputStream <<< " requires objc\n"`
`408`	`407`	`outputStream <<< "}\n"`
`409`	`408`
`410`	`409`	`return outputStream.bytes`
Original file line number	Diff line number	Diff line change
`@@ -584,7 +584,6 @@ fileprivate struct ModuleMapTaskConstructionTests: CoreBasedTests {`
`584`	`584`	`#expect(contents == (OutputByteStream()`
`585`	`585`	`<<< "framework module SwiftOnly {\n"`
`586`	`586`	`<<< " header \"SwiftOnly-Swift.h\"\n"`
`587`		`- <<< " requires objc\n"`
`588`	`587`	`<<< "}\n").bytes)`
`589`	`588`	`}`
`590`	`589`
`@@ -629,7 +628,6 @@ fileprivate struct ModuleMapTaskConstructionTests: CoreBasedTests {`
`629`	`628`	`<<< "\n"`
`630`	`629`	`<<< "module ObjCCompatibilityHeader.Swift {\n"`
`631`	`630`	`<<< " header \"ObjCCompatibilityHeader-Swift.h\"\n"`
`632`		`- <<< " requires objc\n"`
`633`	`631`	`<<< "}\n").bytes)`
`634`	`632`	`}`
`635`	`633`
`@@ -978,7 +976,6 @@ fileprivate struct ModuleMapTaskConstructionTests: CoreBasedTests {`
`978`	`976`	`<<< "\n"`
`979`	`977`	`<<< "module \(targetName).Swift {\n"`
`980`	`978`	`<<< " header \"\(targetName)-Swift.h\"\n"`
`981`		`- <<< " requires objc\n"`
`982`	`979`	`<<< "}\n").bytes)`
`983`	`980`	`}`
`984`	`981`
Original file line number	Diff line number	Diff line change
`@@ -614,7 +614,6 @@ fileprivate struct SwiftTaskConstructionTests: CoreBasedTests {`
`614`	`614`	`#expect(contents == (OutputByteStream()`
`615`	`615`	`<<< "framework module CoreFoo {\n"`
`616`	`616`	`<<< " header \"CoreFoo-Swift.h\"\n"`
`617`		`- <<< " requires objc\n"`
`618`	`617`	`<<< "}\n").bytes)`
`619`	`618`	`}`
`620`	`619`	`}`
`@@ -1094,7 +1093,6 @@ fileprivate struct SwiftTaskConstructionTests: CoreBasedTests {`
`1094`	`1093`	`stream <<< "\n"`
`1095`	`1094`	`stream <<< "module CoreFoo.Swift {\n"`
`1096`	`1095`	`stream <<< " header \"CoreFoo-Swift.h\"\n"`
`1097`		`- stream <<< " requires objc\n"`
`1098`	`1096`	`stream <<< "}\n"`
`1099`	`1097`	`#expect(contents == stream.bytes)`
`1100`	`1098`