Skip to content
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

[main] Enable -user-module-version for 6.0 #7263

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

MaxDesiatov
Copy link
Member

@MaxDesiatov MaxDesiatov commented Jan 17, 2024

Summary:
#if canImport(LibFoo, _version: "1.2.3") is not currently supported by SwiftPM, as this feature was gated on .vNext, even though it landed before SwiftPM 5.10 branched off.

Steps To Reproduce:

  1. Build a package with // swift-tools-version: 6.0 and try to use #if canImport(LibFoo, _version: "1.2.3") for a dependency LibFoo.

Results:
The availability check won't work because -user-module-version option is not passed correctly to the build system.

This was gated on `.vNext` previously, even though it was introduced before 5.10 branched off. It feels safe to enable this in 5.10.
@MaxDesiatov
Copy link
Member Author

@swift-ci test

1 similar comment
@MaxDesiatov
Copy link
Member Author

@swift-ci test

@MaxDesiatov MaxDesiatov changed the title Enable -user-module-version for 5.10 [main] Enable -user-module-version for 5.10 Jan 18, 2024
@MaxDesiatov MaxDesiatov changed the title [main] Enable -user-module-version for 5.10 [main] Enable -user-module-version for 5.11 Jan 19, 2024
@MaxDesiatov
Copy link
Member Author

@swift-ci test

@MaxDesiatov
Copy link
Member Author

@swift-ci test windows

@MaxDesiatov MaxDesiatov added the needs tests This change needs test coverage label Mar 4, 2024
@MaxDesiatov MaxDesiatov changed the title [main] Enable -user-module-version for 5.11 [main] Enable -user-module-version for 6.0 Mar 4, 2024
@MaxDesiatov
Copy link
Member Author

@swift-ci test

@bnbarham
Copy link
Contributor

@swift-ci please test Windows platform

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug DO NOT MERGE needs tests This change needs test coverage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants