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

[vcpkg] Put user-defined configuration triplets path on top of the search queue #11931

Merged

Conversation

AenBleidd
Copy link
Contributor

I believe that user-defined configuration triplets paths should be the first when searching for triplet configuration.
This commit puts user-defined configuration triplets (if defined) on top of the search queue before standard paths.
This could help to overwrite default tiplet configuration without setting custom triplet.

P.S. If current implementation is better than my suggestion, feel free to ignore this commit.

Signed-off-by: Vitalii Koshura [email protected]

I believe that user-defined configuration triplets paths should be the first when searching for triplet configuration.
This commit puts user-defined configuration triplets (if defined) on top of the search queue before standard paths.
This could help to overwrite default tiplet configuration without setting custom triplet.

P.S. If current implementation is better than my suggestion, feel free to ignore this commit.

Signed-off-by: Vitalii Koshura <[email protected]>
@AenBleidd AenBleidd changed the title Put user-defined configuration triplets path on top of the search queue [vcpkg] Put user-defined configuration triplets path on top of the search queue Jun 14, 2020
@LilyWangL LilyWangL requested a review from vicroms June 15, 2020 06:08
@LilyWangL LilyWangL added category:vcpkg-feature The issue is a new capability of the tool that doesn’t already exist and we haven’t committed requires:discussion labels Jun 15, 2020
@LilyWangL LilyWangL added the info:reviewed Pull Request changes follow basic guidelines label Jun 18, 2020
@vicroms
Copy link
Member

vicroms commented Jun 18, 2020

Thanks for the PR! This was actually a bug in vcpkg 😨

@vicroms vicroms merged commit de44a80 into microsoft:master Jun 18, 2020
@AenBleidd AenBleidd deleted the vko_put_user_defined_triplets_path_on_top branch June 19, 2020 08:52
strega-nil pushed a commit to strega-nil/vcpkg that referenced this pull request May 5, 2021
…ue (microsoft#11931)

I believe that user-defined configuration triplets paths should be the first when searching for triplet configuration.
This commit puts user-defined configuration triplets (if defined) on top of the search queue before standard paths.
This could help to overwrite default tiplet configuration without setting custom triplet.

P.S. If current implementation is better than my suggestion, feel free to ignore this commit.

Signed-off-by: Vitalii Koshura <[email protected]>
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 info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants