Skip to content

[6.0][PubGrub] Avoid resolving package versions twice in presence of prebu… #7457

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

Conversation

xedin
Copy link
Contributor

@xedin xedin commented Apr 12, 2024

…ilt libraries

  • Explanation:

Integrate information about versions of prebuilt libraries into the solver and prioritize these versions when making decisions if they meet all of the requirements of undecided terms.

This helps solver to avoid fetching and at the same time allows partial matches on prebuilt libraries with fallback to package provided versions.

(cherry picked from commit 6eb042d)
(cherry picked from commit 0e897ed)
(cherry picked from commit ee6a58d)
(cherry picked from commit b817df1)
(cherry picked from commit 2f48249)

…ilt libraries

Integrate information about versions of prebuilt libraries into the solver
and prioritize these versions when making decisions if they meet all of the
requirements of undecided terms.

This helps solver to avoid fetching and at the same time allows partual
matches on prebuilt libraries with fallback to package provided versions.

(cherry picked from commit 6eb042d)
(cherry picked from commit 0e897ed)
(cherry picked from commit ee6a58d)
(cherry picked from commit b817df1)
(cherry picked from commit 2f48249)
@xedin xedin added swift 6.0 Related to Swift 6.0 release branch dependency resolution labels Apr 12, 2024
@xedin
Copy link
Contributor Author

xedin commented Apr 12, 2024

@swift-ci please test

@xedin xedin merged commit 04397a4 into swiftlang:release/6.0 Apr 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependency resolution swift 6.0 Related to Swift 6.0 release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants