Skip to content

Synchronize gdextension_interface.h with Godot #1117

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 1 commit into from
May 26, 2023

Conversation

dsnopek
Copy link
Collaborator

@dsnopek dsnopek commented May 17, 2023

At the moment, the gdextension_interface.h in godot-cpp doesn't match the gdextension_interface.h in Godot.

These getting out-of-sync was my fault: with all the rebasing I did around godotengine/godot#35813 and godotengine/godot#76406, the version of gdextension_interface.h that ended up getting committed to godot-cpp is somehow missing the changes from godotengine/godot#35813.

But it's an easy fix :-)

@dsnopek dsnopek added the bug This has been identified as a bug label May 17, 2023
@dsnopek dsnopek requested a review from a team as a code owner May 17, 2023 18:18
@Calinou Calinou added topic:gdextension This relates to the new Godot 4 extension implementation bug This has been identified as a bug and removed bug This has been identified as a bug labels May 17, 2023
@BastiaanOlij
Copy link
Collaborator

We should probably add something a little bit more clearer that people should be using the appropriate tag when they are working on an extension for the current stable release of Godot.

We should indeed update master here as soon as this file changes on the Godot side.

@dsnopek dsnopek force-pushed the sync-gdextension-interface branch from 7ad0bf3 to 6347ce2 Compare May 26, 2023 20:45
@dsnopek
Copy link
Collaborator Author

dsnopek commented May 26, 2023

We should probably add something a little bit more clearer that people should be using the appropriate tag when they are working on an extension for the current stable release of Godot.

Some text to that effect was added in #1115 - it's kind of specific to 4.0 vs 4.1, but it's good for now!

@dsnopek dsnopek merged commit 0c466dd into godotengine:master May 26, 2023
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 topic:gdextension This relates to the new Godot 4 extension implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants