Skip to content

Conversation

@niklas2902
Copy link

Hey 🙂,

I’m currently trying to get a better understanding of godot-python to see if I can help push development forward a bit.
I noticed that saving files didn’t seem to work. With this fix, it should. But I’m not entirely sure if this is the right way to do it 🙈.

If it doesn’t help, feel free to just close it.

@touilleMan
Copy link
Owner

Hi @niklas2902 , thanks for the patch it seems good enough to me ;-)

To be honest the PythonResourceFormatSaver class is just a mock for the moment, I'm focusing on PythonResourceFormatLoader&PythonScriptLanguage for the moment

If you are curious about what I'm currently working on: I was doing some testing to determine if it was worth it to entirely ditch the ScriptLanguage system and instead only use GDExtension classes to expose Python classes, unfortunately didn't turn out great since the GDExtension still lacks some key features such as the possibility to load a class from its path (GDExtension requires each exposed class to be explicitly registered to Godot, typically when the extension is loaded, but this is a cumbersome step when writing scripts, and it means there is no simple way to skip some script loading if only a subset is needed, for instance when running a CLI command, which is a typical usecase for Python ^^)

@touilleMan touilleMan merged commit 17aa4fe into touilleMan:godot4-meson Nov 17, 2025
0 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants