search

GreenCrowDev / hoodie

Hoodie is a plugin for Godot Engine 4 that offers a visual scripting interface for generating procedural geometry.
MIT License
325 stars 14 forks source link

Release 0.1 zipfile might lack a DLL or a tweak #5

Closed Eiyeron closed 7 months ago

Eiyeron commented 7 months ago

Hello!

I look forward your progress with this project, it got my interest pretty quickly and I tried to test it in a blank new project.

In both Godot 4.2.1 and 4.3-dev4, I couldn't load the extension directly from the content of the release zip. I had to copy libhoodie.windows.template_release.x86_64.dll to libhoodie.windows.template_debug.x86_64.dll in order for the extension to load. I guess editing the gdextension file would have been simpler.

Anyhow, once the tweak was done, the extension worked well and I could confirm that it'd work fine.

Have a nice day!

GreenCrowDev commented 7 months ago

Hello!

Thank you for the report! I updated the release with the template_debug version of the .dll in order for the extension to load. I'm probably still misunderstanding something about target templates in Godot Engine, but I'm figuring it out right now. I'll probably have to update it again soon with a proper way to do it. In the meantime, it should work.

Have a nice day! 😊

Eiyeron commented 7 months ago

Thanks for the quick answer. Now it does work in the editor when dropped as is.

I wonder if you should bundle all the DLLs to cover all the possible use cases in the development lifecycle, I suppose (with absolutely zero checking in the doc) that the release DLL might be still used, but only for exported projects. I'm not able to confirm my hypothesis right now, but that's something I'll try to keep in mind to test as soon as possible.

Have a nice day!

GreenCrowDev commented 7 months ago

Yeah, I got some help on the Godot Discord, and probably the best thing to do in the end is pack all the DLLs.

The release version seems in fact only for exported projects, but I haven't tested that yet, since I still don't even have a use case for that.

Thanks for the report, let me know when you find some other bug to squash!