godot-git-plugin icon indicating copy to clipboard operation
godot-git-plugin copied to clipboard

Update godot-cpp to the latest on the `4.1` branch

Open dsnopek opened this issue 1 year ago • 1 comments
trafficstars

This updates the godot-cpp used by this extension to the latest on godot-cpp's 4.1 branch.

This will mean that the extension will only be compatible with Godot 4.1.4 or later due to the updates to the extension_api.json. However, if it was important than it still worked with Godot 4.1.0 to 4.1.3, then it should be possible to compile with scons custom_api_file=path/to/extension_api.json where that JSON file is from Godot 4.1.0 - and we could even stash that extension_api.json in the repo? Anyway, just let me know if you'd like adjustments for that.

However, updating to the newest godot-cpp that will work with Godot 4.1 will include a whole bunch of bug fixes that have been cherry-picked since then.

For example, PR https://github.com/godotengine/godot-cpp/pull/1313 which fixes https://github.com/godotengine/godot-git-plugin/issues/239

As well as other various memory leaks, crashes, build-system improvements, etc.

dsnopek avatar Jul 10 '24 16:07 dsnopek

Hm, not sure why SCons is failing on the CI, it's working for me locally...

dsnopek avatar Jul 10 '24 16:07 dsnopek

Per conversation on RocketChat, I've switched this to updating to godot-cpp's godot-4.2-stable tag (rather than the latest on the 4.1 branch)

Also, I've switched this to DRAFT because it includes the changes from PR https://github.com/godotengine/godot-git-plugin/pull/272 so that the CI will pass

dsnopek avatar Mar 24 '25 14:03 dsnopek

#272 is merged, so this can be rebased.

akien-mga avatar May 25 '25 13:05 akien-mga

Rebased!

dsnopek avatar May 26 '25 13:05 dsnopek

The commit still mentions 4.1, this should likely be updated.

akien-mga avatar May 26 '25 13:05 akien-mga

Ack, sorry! Commit message is fixed in latest push

dsnopek avatar May 26 '25 14:05 dsnopek

Thanks!

akien-mga avatar May 26 '25 14:05 akien-mga