Skip to content

Fixes package import stmt missing from .package.swiftinterface. #72282

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
merged 1 commit into from
Mar 14, 2024

Conversation

elsh
Copy link
Contributor

@elsh elsh commented Mar 13, 2024

Resolves rdar://124309585

@elsh elsh requested review from artemcm and tshortli as code owners March 13, 2024 01:24
@elsh elsh requested review from xymus and nkcsgexi March 13, 2024 01:24
@elsh
Copy link
Contributor Author

elsh commented Mar 13, 2024

@swift-ci smoke test

@elsh elsh force-pushed the elsh/pkg-import-pkg-interface branch from 8ff9ef8 to e9cab7a Compare March 13, 2024 08:27
@elsh
Copy link
Contributor Author

elsh commented Mar 13, 2024

@swift-ci smoke test

@elsh elsh force-pushed the elsh/pkg-import-pkg-interface branch from e9cab7a to 2d90423 Compare March 13, 2024 21:37
@elsh
Copy link
Contributor Author

elsh commented Mar 13, 2024

@swift-ci smoke test

@@ -332,7 +349,7 @@ static void printImports(raw_ostream &out,
out << "@_implementationOnly ";
}

if (publicImportSet.count(import))
if (exportedImportSet.count(import))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That rename was long overdue, thanks!

@elsh elsh enabled auto-merge March 13, 2024 22:18
@elsh elsh merged commit 378884e into main Mar 14, 2024
@elsh elsh deleted the elsh/pkg-import-pkg-interface branch March 14, 2024 02:35
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.

2 participants