Skip to content

[vcpkg] Implement constraints and overrides in manifests#14183

Merged
ras0219-msft merged 6 commits intomicrosoft:masterfrom
ras0219:dev/roschuma/versioning
Nov 13, 2020
Merged

[vcpkg] Implement constraints and overrides in manifests#14183
ras0219-msft merged 6 commits intomicrosoft:masterfrom
ras0219:dev/roschuma/versioning

Conversation

@ras0219
Copy link
Contributor

@ras0219 ras0219 commented Oct 23, 2020

This PR implements parsing of constraints and overrides for manifest files as specified in #11758.

It includes guard logic to prevent accidental usage of these new features in manifest files while the experimental versioning feature flag is not specified; this ensures users will not be broken as we continue to develop.

See especially the tests in toolsrc/src/vcpkg-test/manifests.cpp for exactly compiling examples.

@JackBoosY JackBoosY added category:vcpkg-feature The issue is a new capability of the tool that doesn’t already exist and we haven’t committed info:internal labels Oct 26, 2020
@vicroms
Copy link
Member

vicroms commented Nov 12, 2020

@ras0219 is there any work left here?

@ras0219-msft ras0219-msft merged commit 7837631 into microsoft:master Nov 13, 2020
strega-nil pushed a commit to strega-nil/vcpkg that referenced this pull request May 5, 2021
…4183)

* [vcpkg] Implement constraints in manifests

* [vcpkg] Add SourceControlFile::check_against_feature_flags to prevent accidentally ignoring versioning fields

* [vcpkg] Switch check_against_feature_flags to accept fs::path

* [vcpkg] Implement overrides parsing in manifests

* [vcpkg] Address CR comments

Co-authored-by: Robert Schumacher <roschuma@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:vcpkg-feature The issue is a new capability of the tool that doesn’t already exist and we haven’t committed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants