Skip to content

[SwiftCore] Enable explicit module builds #80354

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

etcwilde
Copy link
Member

Explicit module builds help prevent the dependency scanner from getting lost and attempting to pull the SwiftShims module from the resource directory, existing SDK, and the just-built standard library. They aren't part of the library interface, but make it possible to build the runtime libraries without seeing duplicate shims. Moving the flag to a top-level compile command.

Explicit module builds help prevent the dependency scanner from getting
lost and attempting to pull the SwiftShims module from the resource
directory, existing SDK, and the just-built standard library.
They aren't part of the library interface, but make it possible to build
the runtime libraries without seeing duplicate shims. Moving the flag to
a top-level compile command.
@etcwilde
Copy link
Member Author

@al45tair, this should fix the duplicate module issue you were seeing before.

@etcwilde
Copy link
Member Author

@swift-ci please smoke test

@etcwilde
Copy link
Member Author

@swift-ci please smoke test macOS

@al45tair
Copy link
Contributor

main/swift/Runtimes/Supplemental/StringProcessing/RegexBuilder/CharacterClass.swift:12:17: error: Unable to find module dependency: '_RegexParser'
internal import _RegexParser
                ^
/Users/ec2-user/jenkins/workspace/swift-PR-macos-smoke-test/branch-main/swift/Runtimes/Supplemental/StringProcessing/RegexBuilder/CharacterClass.swift:12:17: note: a dependency of main module 'RegexBuilder'
internal import _RegexParser
                ^
/Users/ec2-user/jenkins/workspace/swift-PR-macos-smoke-test/branch-main/swift/Runtimes/Supplemental/StringProcessing/RegexBuilder/Anchor.swift:12:17: note: also imported here
internal import _RegexParser
                ^
/Users/ec2-user/jenkins/workspace/swift-PR-macos-smoke-test/branch-main/swift/Runtimes/Supplemental/StringProcessing/RegexBuilder/DSL.swift:12:17: note: also imported here
internal import _RegexParser
                ^
error: Unable to find module dependency: '_RegexParser'

Looks like maybe the Regex code isn't compatible with explicit modules for some reason.

@etcwilde
Copy link
Member Author

Yeah, that should be fixed by #80362.

@etcwilde
Copy link
Member Author

@swift-ci please smoke test macOS

@etcwilde etcwilde merged commit 2a7dbd8 into swiftlang:main Mar 29, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants