Skip to content

Bump swift-crypto to 3.0.0 in Package.swift #904

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
Oct 16, 2023

Conversation

joehsieh
Copy link
Contributor

To address the error

Dependencies could not be resolved because root depends on 'swift-crypto' 2.5.0..<3.0.0 and 'swift-package-manager' 
depends on 'swift-crypto' 3.0.0..<3.1.0. 

related issue: #903

@@ -270,7 +270,7 @@ if ProcessInfo.processInfo.environment["SWIFTCI_USE_LOCAL_DEPS"] == nil {
.package(url: "https://github.com/apple/swift-tools-support-core.git", branch: relatedDependenciesBranch),
.package(url: "https://github.com/apple/swift-argument-parser.git", from: "1.2.2"),
.package(url: "https://github.com/apple/swift-syntax.git", branch: relatedDependenciesBranch),
.package(url: "https://github.com/apple/swift-crypto.git", from: "2.5.0"),
.package(url: "https://github.com/apple/swift-crypto.git", from: "3.0.0"),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

[Question] what's the logic to decide the range of versions?
should we align it with the setting in swift-package-manager
.upToNextMinor(from: "3.0.0")

Copy link
Contributor

@MaxDesiatov MaxDesiatov Oct 16, 2023

Choose a reason for hiding this comment

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

This one being broad is fine, means we'll only need to bump it in SwiftPM in the future for minor releases and keep it here as is until 4.0

@MaxDesiatov
Copy link
Contributor

@swift-ci test

@ahoppen
Copy link
Member

ahoppen commented Oct 16, 2023

Thank you for fixing the issue @joehsieh!

@ahoppen ahoppen merged commit ec2e3c1 into swiftlang:main Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants