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

[kf5solid] Fix baseline regression, cleanup #34085

Merged
merged 6 commits into from
Oct 3, 2023

Conversation

dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Sep 28, 2023

Amends #33888.

The portfile didn't pass FEATURE_OPTIONS to the CMake build, allowing to find optional dependencies when they weren't selected. This breaks later builds from cached artifact as in vcpkg CI https://dev.azure.com/vcpkg/public/_build/results?buildId=94631.

@dg0yt dg0yt changed the title [f5solid] Fix baseline regression, cleanup [kf5solid] Fix baseline regression, cleanup Sep 28, 2023
@JonLiu1993 JonLiu1993 self-assigned this Sep 28, 2023
@JonLiu1993 JonLiu1993 added the category:port-bug The issue is with a library, which is something the port should already support label Sep 28, 2023
@BillyONeal BillyONeal merged commit cc57622 into microsoft:master Oct 3, 2023
15 checks passed
@BillyONeal
Copy link
Member

Thanks!

@dg0yt dg0yt deleted the kf5solid branch October 3, 2023 05:35
clementperon pushed a commit to clementperon/vcpkg that referenced this pull request Oct 16, 2023
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants