Skip to content

Test compatibility updates for Swift Build BSP backend - #2741

Draft
owenv wants to merge 1 commit into
swiftlang:mainfrom
owenv:owenv/compattweaks
Draft

Test compatibility updates for Swift Build BSP backend#2741
owenv wants to merge 1 commit into
swiftlang:mainfrom
owenv:owenv/compattweaks

Conversation

@owenv

@owenv owenv commented Aug 4, 2026

Copy link
Copy Markdown
Contributor
  • Update an progress message assertion which differs slightly between build system backends
  • Update an output path assertion which differs between backends
  • Update two tests to build before making cross-module definition requests in an Objective-C package. Previously, the native build system would construct module maps at build planning time. Swift Build generates these during the build (or preparation build). This is a behavioral change, but the new behavior is in general much better for performance and the correctness of incremental builds, so I don't think we want to try applying some hack to avoid the build/preparation here.

@owenv

owenv commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

@swift-ci test

@owenv

owenv commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

There's some inconsistency in whether definition requests return a range or position depending on if the index is populated I need to sort out here

@owenv
owenv marked this pull request as draft August 4, 2026 16:23
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.

1 participant