Skip to content

Conversation

BillyONeal
Copy link
Member

@BillyONeal BillyONeal commented Apr 30, 2025

Before:

.\vcpkg.exe install vcpkg-depends-on-fail[x] --overlay-ports C:\Dev\vcpkg-tool\azure-pipelines\e2e-ports --keep-going --binarysource clear
[...]
error: building vcpkg-depends-on-fail:x64-windows failed with: CASCADED_DUE_TO_MISSING_DEPENDENCIES
  due to the following missing dependencies:
    vcpkg-fail-if-depended-upon[a]:x64-windows
    vcpkg-fail-if-depended-upon[core]:x64-windows
    vcpkg-fail-if-depended-upon[b]:x64-windows
    vcpkg-fail-if-depended-upon[core]:x64-windows

After:

.\vcpkg.exe install vcpkg-depends-on-fail[x] --overlay-ports C:\Dev\vcpkg-tool\azure-pipelines\e2e-ports --keep-going --binarysource clear
[...]
error: building vcpkg-depends-on-fail:x64-windows failed with: CASCADED_DUE_TO_MISSING_DEPENDENCIES
  due to the following missing dependencies:
    vcpkg-fail-if-depended-upon[a,b,core]:x64-windows

I discovered this upon seeing that there is a duplicate implementation here I'm removing:

std::string failed_dependencies;

Building vcpkg-depends-on-fail[core,x]:$($Triplet)@0...
error: building vcpkg-depends-on-fail:$($Triplet) failed with: CASCADED_DUE_TO_MISSING_DEPENDENCIES
due to the following missing dependencies:
vcpkg-fail-if-depended-upon[a,b,core]:$($Triplet)
Copy link
Member Author

Choose a reason for hiding this comment

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

Note that this does not list is-a-default-feature even though we tried to install it here because that isn't actually a dependency of vcpkg-depends-on-fail

@BillyONeal BillyONeal requested a review from JavierMatosD May 1, 2025 19:18
@BillyONeal BillyONeal enabled auto-merge (squash) May 1, 2025 19:54
@BillyONeal BillyONeal merged commit d0e1cbb into microsoft:main May 1, 2025
7 checks passed
@BillyONeal BillyONeal deleted the merge-cascade-failures branch May 1, 2025 23:57
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.

2 participants