-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
[godot-cpp] Update to 4.3 #41246
[godot-cpp] Update to 4.3 #41246
Conversation
-cmake_minimum_required(VERSION 3.12) | ||
-cmake_minimum_required(VERSION 3.13) | ||
+cmake_minimum_required(VERSION 3.19) |
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.
Why does this need to be patched at all?
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.
This change from PR #37931 which creates this port.
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.
I know, but the change to 3.13 is the upstreamed change from the initial porting.
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.
(Note: I asked why it is necessary, not when it was added.)
Update
godot-cpp
to 4.3.Usage passed on
x64-windows
.The "supports" clause reflects platforms that may be fixed by this new version.Any fixed CI baseline entries are removed from that file.Any patches that are no longer applied are deleted from the port's directory.When updating the upstream version, the"port-version"
is reset (removed fromvcpkg.json
)../vcpkg x-add-version --all
and committing the result.