Skip to content

[many ports] Add supports#12857

Merged
BillyONeal merged 7 commits intomicrosoft:masterfrom
PhoebeHui:dev/Phoebe/add_supports
Sep 3, 2020
Merged

[many ports] Add supports#12857
BillyONeal merged 7 commits intomicrosoft:masterfrom
PhoebeHui:dev/Phoebe/add_supports

Conversation

@PhoebeHui
Copy link
Contributor

Closes #12390 #12830 #11277 (comment)

Add unsupport arch/target to 'Support".

@PhoebeHui PhoebeHui added category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:internal labels Aug 11, 2020
@Neumann-A
Copy link
Contributor

Add unsupport arch/target to 'Support".

@ras0219 @ras0219-msft @strega-nil : I think the Supports field should only be used if a port is not designed to be used on a specific platform and not mean vcpkg is just not currently able to build the port on that platform. Maybe a clear guideline when /how to use the Supports field is required.

@BillyONeal
Copy link
Member

We had an IRL discussion about this on Thursday. The results of that were:

  • The Supports field is primarily intended to avoid folks needing to waste time building stuff that we know will be broken. To that end, it is an artifact of the port, not of the package, so if the port we have is unable to target a platform it doesn't matter what the package supports.
  • The Supports field is not an artifact of our CI system, so if a port works locally but is broken in our CI system for some reason (e.g. conflicts with another port in a way we accept like boringssl, depends on some dependency already being installed we can't install there for some reason, etc) that goes in ci.baseline.txt, not Supports.

I believe this change is in keeping with those terms and thus good.

…orts

# Conflicts:
#	ports/speex/CONTROL
#	ports/speex/portfile.cmake
@PhoebeHui
Copy link
Contributor Author

The I have added the !static to Supports for the 2 ports, however, it doesn't skip in CI, still failed here.

REGRESSION: alembic:x64-windows-static. If expected, add alembic:x64-windows-static=fail to .\scripts\ci.baseline.txt.
REGRESSION: avisynthplus:x64-windows-static. If expected, add avisynthplus:x64-windows-static=fail to .\scripts\ci.baseline.txt.

@BillyONeal
Copy link
Member

The I have added the !static to Supports for the 2 ports, however, it doesn't skip in CI, still failed here.

vcpkg ci x64-windows-static for some reason isn't turning on the "static" flag when evaluating "supports" expressions. :(

@BillyONeal BillyONeal merged commit e7dc93f into microsoft:master Sep 3, 2020
@BillyONeal
Copy link
Member

Thanks for your contribution!

@PhoebeHui PhoebeHui deleted the dev/Phoebe/add_supports branch October 22, 2020 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vcpkg install atkmm error

5 participants