Skip to content

🍒 [6.1] Indexing: add blocklist support for avoiding indexing specific module names #79064

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
Jan 31, 2025

Conversation

nkcsgexi
Copy link
Contributor

  • Explanation: Indexing while building sometimes triggers module deserialization issues, exemplified by a recent issue of rdar://141357099. This change introduces the blocklist support to avoid indexing specific module names so we could rely on external data source for unblocking builds, instead of modifying the compiler source.

  • Scope: Indexing while building

  • Risk: Very Low.

  • Testing: Manually tested with actual project that enables indexing-while-building

  • Cherry-picked PR: Indexing: add blocklist support for avoiding indexing specific module names #78984

  • Reviewer: @artemcm @bnbarham

  • Issue: rdar://143770366

@nkcsgexi nkcsgexi requested a review from bnbarham January 30, 2025 22:06
@nkcsgexi nkcsgexi requested a review from a team as a code owner January 30, 2025 22:06
@nkcsgexi
Copy link
Contributor Author

@swift-ci please smoke test

@nkcsgexi
Copy link
Contributor Author

@swift-ci please test

@nkcsgexi nkcsgexi requested a review from artemcm January 30, 2025 22:08
… names

Indexing while building sometimes triggers module deserialization issues, exemplified
by a recent issue of rdar://141357099. This change introduces the blocklist support
to avoid indexing specific module names so we could rely on external data source for
unblocking builds, instead of modifying the compiler source.

Resolves: rdar://143770366
@nkcsgexi nkcsgexi force-pushed the block-list-module-index branch from 7d6a01e to f06660a Compare January 31, 2025 00:47
@nkcsgexi
Copy link
Contributor Author

@swift-ci please test

Copy link
Contributor

@bnbarham bnbarham left a comment

Choose a reason for hiding this comment

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

Thanks, please change on main too 🙏

@nkcsgexi nkcsgexi merged commit 7ea1c69 into release/6.1 Jan 31, 2025
5 checks passed
@nkcsgexi nkcsgexi deleted the block-list-module-index branch January 31, 2025 20:56
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