Skip to content

Community Triplets 🤝#7976

Merged
vicroms merged 7 commits intomicrosoft:masterfrom
vicroms:pr/community-triplets
Jan 17, 2020
Merged

Community Triplets 🤝#7976
vicroms merged 7 commits intomicrosoft:masterfrom
vicroms:pr/community-triplets

Conversation

@vicroms
Copy link
Member

@vicroms vicroms commented Aug 29, 2019

Add a new folder <vcpkg-root>/triplets/community where commonly requested triplets will be hosted.

Community Triplets, unlike regular triplets, won't be tested during CI. This means that no regressions will be reported for code changes that break compatibility with community triplets.

These triplets are disabled by default but can be enabled by passing the --overlay-triplets=<path-to-vcpkg>/triplets/community option in your commands.

@vicroms vicroms self-assigned this Aug 29, 2019
@vicroms
Copy link
Member Author

vicroms commented Aug 30, 2019

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@cbezault cbezault left a comment

Choose a reason for hiding this comment

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

LGTM, my one comment is more of a nit than anything.

for (auto&& path : fs.get_files_non_recursive(triplets_dir))
{
output.push_back(path.stem().filename().string());
if (fs::is_regular_file(fs.status(VCPKG_LINE_INFO, path)))
Copy link
Contributor

Choose a reason for hiding this comment

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

I wish the error that this output made it more obvious that it explicitly was the triplet path that could not be found.

@JackBoosY
Copy link
Contributor

/azp run

@vicroms vicroms marked this pull request as ready for review January 17, 2020 22:20
@vicroms vicroms merged commit f56645c into microsoft:master Jan 17, 2020
kevinlul added a commit to kevinlul/edopro-vcpkg-cache that referenced this pull request Jan 22, 2020
@vicroms vicroms deleted the pr/community-triplets branch October 27, 2020 06:11
strega-nil pushed a commit to strega-nil/vcpkg that referenced this pull request May 5, 2021
* Move untested triplets to Community Triplets

* Document community triplets

* Load community triplets by default and alert when using one

* [triplets] Merge documentation of community triplets

* Bump version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants