-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
[Thorvg] Update Thorvg port to v0.10.0 #32934
Conversation
ports/thorvg/vcpkg.json
Outdated
"description": "ThorVG is a platform-independent portable library for drawing vector-based scenes and animations", | ||
"homepage": "https://www.thorvg.org", | ||
"license": "MIT", | ||
"supports": "!(arm & uwp)", | ||
"supports": "!(arm & uwp) & !windows", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rinechran, thanks for the PR, could you explain why the new version doesn't support the windows platform?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is an error in the current Windows build.
For this, I am planning to solve the problem and update it by pring it directly to Thorvg.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ports that do not support certain triples in vcpkg for the time being can write them to the baseline, please add
thorvg:x86-windows=fail
thorvg:x64-windows=fail
thorvg:x64-windows-static=fail
thorvg:x64-windows-static-md=fail
under this:
Line 1306 in 1e9cf69
thorvg:x64-android=fail |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And if this port doesn't support some triplets, please add supports keyword in vcpkg.json instead of modify baseline.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I found the cause and I'm going to patch it up now and upload it again
I made a mistake in the review
Note: I will be converting your PR to draft status. When you respond, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags. |
Feature |
./vcpkg x-add-version --all
and committing the result.