Skip to content

Cherry-picks for the godot-cpp 4.4 branch - 2nd batch #1805

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

Merged
merged 41 commits into from
Jul 3, 2025

Conversation

dsnopek
Copy link
Collaborator

@dsnopek dsnopek commented Jun 24, 2025

The 2nd batch of PR's marked with cherrypick:4.4

The was the biggest cherry-picking batch I've ever done - I waited way too long to do it :-)

dsnopek and others added 30 commits March 27, 2025 07:31
(cherry picked from commit c963321)
(cherry picked from commit 946a693)
- Duplicated the ci.yml into ci-scons.yml and ci-cmake.yml
- Replaced some manual shell commands to detect changes with  tj-actions/changed-files@v45
- Conditionally run the CI depending on what files change.
- Add ios toolchain from leetal/ios-cmake
- Added a cmake-minimum of 3.10 to silence errors from ios toolchain

(cherry picked from commit b6c0251)
(cherry picked from commit 552a74c)
(cherry picked from commit 38e517b)
This is required when publishing to the App Store, and consistent with
what we already do for the macOS deployment target.

(cherry picked from commit 4879eb7)
This was referenced Jun 24, 2025
(cherry picked from commit 8938e7e)
Copy link
Member

@vnen vnen left a comment

Choose a reason for hiding this comment

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

At a glance everything is alright. I'll trust @dsnopek on this one.

Copy link
Member

@aaronfranke aaronfranke left a comment

Choose a reason for hiding this comment

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

Tested on Godot 4D and it compiles as-is, and runs with both Godot 4.4.1 and master (4.5.beta). Note that Godot 4.4.0 did not work, it gave an error that 4.4.1 or later is required.

@dsnopek
Copy link
Collaborator Author

dsnopek commented Jul 3, 2025

@aaronfranke Thanks for testing!

Note that Godot 4.4.0 did not work, it gave an error that 4.4.1 or later is required.

This would also be the case using the 4.4 branch without this PR, due to e4b7c25

If you want to continue to support 4.4.0+, then you'll need to build using the extension_api.json from 4.4.0

@dsnopek dsnopek merged commit d502d8e into godotengine:4.4 Jul 3, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This has been identified as a bug enhancement This is an enhancement on the current functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.