Skip to content

Conversation

@bnbarham
Copy link
Contributor

@bnbarham bnbarham commented Dec 2, 2025

Also adds some logging to the synchronize request so that it's obvious when background indexing is being skipped.

Fixes #2373
Resolves rdar://165519940

@bnbarham
Copy link
Contributor Author

bnbarham commented Dec 2, 2025

@swift-ci please test

@bnbarham bnbarham force-pushed the pass-toolset-through branch from 0ee3a7d to f8c3779 Compare December 2, 2025 06:28
@bnbarham
Copy link
Contributor Author

bnbarham commented Dec 2, 2025

@swift-ci please test

Copy link
Member

@ahoppen ahoppen left a comment

Choose a reason for hiding this comment

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

Oh, good catch. Completely missed this in #2058 🙈

@ahoppen
Copy link
Member

ahoppen commented Dec 2, 2025

@swift-ci Please test Windows

@bnbarham bnbarham force-pushed the pass-toolset-through branch from f8c3779 to a36a2f2 Compare December 2, 2025 11:41
@bnbarham
Copy link
Contributor Author

bnbarham commented Dec 2, 2025

@swift-ci please test

@bnbarham
Copy link
Contributor Author

bnbarham commented Dec 2, 2025

@swift-ci please test Windows platform

Also adds some logging to the synchronize request so that it's obvious
when background indexing is being skipped.

Fixes swiftlang#2373
Resolves rdar://165519940
@bnbarham bnbarham force-pushed the pass-toolset-through branch from a36a2f2 to 9952a11 Compare December 3, 2025 00:20
@bnbarham
Copy link
Contributor Author

bnbarham commented Dec 3, 2025

@swift-ci please test

@bnbarham
Copy link
Contributor Author

bnbarham commented Dec 3, 2025

Updated the test to include checking options and skipping on Windows due to swiftlang/swift-package-manager#9438.

@bnbarham
Copy link
Contributor Author

bnbarham commented Dec 3, 2025

@swift-ci please test Windows platform

@bnbarham bnbarham merged commit 9c145fd into swiftlang:main Dec 3, 2025
3 checks passed
@bnbarham bnbarham deleted the pass-toolset-through branch December 3, 2025 21:46
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.

[Embedded] Can't find any module when background indexing is turned on

2 participants