-
Notifications
You must be signed in to change notification settings - Fork 68
-
Notifications
You must be signed in to change notification settings - Fork 68
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
Godex doesn't compile as godot module #176
Comments
Hey, Can you please share some more info on how your project is setup? Maybe related to: #158 |
Hey @AndreaCatania. Sorry about that. Let me clarify: I believe it is compiled as a module. I followed the instructions found here. Is there a different way I should set this up? #158 claims the PR #159 is already merged in. I did run this under master for both Godot and Godex. Should I try an external module install? Thanks. |
No that's absolutely fine, I was just asking. Sorry for the delay but I'm improving the script systems and bundle usage, so I been a bit busy. If you can try to compile this as external module it would be useful (since this is the way I'm using it, and the CI is using it). So, we will know for sure if the problem is related to how the Godex module is imported and so if #158 is still relevant. This will allow me to dig into the problem and eventually fix it. To compile Godex as external module you have to:
You can find more info on the custom modules here |
similar problem for me on linux:
running |
@matthew-salerno perfect thanks! |
I get the same issue
I attempted following these steps and running
|
It looks like the patch wasn't properly applied, since those methods are missing: https://github.com/GodotECS/godex/blob/main/patches/add_custom_iterator.patch#L29-L32 |
Just to add more info, when trying to compile on Windows as a module in the current commit, the build fails with
If I change the 'godex/ecs_types.h' include to 'modules/godex/ecs_types.h' the compiling continues until it breaks with the same message on the opening post of the issue. Compiling godex as a custom module ( |
I just had the same problem as @fnzr, except I'm on Linux; however, I can verify that compiling godotengine/godot@5f69218 (current) with Godex specified as a custom module works fine. Relatedly, @AndreaCatania, I forked the wiki and drastically updated the setup page, as you can see here. I think it's much easier for new users to follow. It also adds the To my knowledge, I cannot submit a PR for the wiki, so you may want to see these instructions on merging wikis if you think my setup page is an improvement. |
@reeseschultz Wow, the new Setup wiki is really nice, thanks a lot! :) :) |
@AndreaCatania, you're welcome, but I don't think the code blocks in your update to the setup page are quite right. It looks like you may be trying to fit the ``` blocks on a single line, and if that's the case, you'll want to do just a single `. I see this in all the summary sections. Anyway, I'll let you know if I update the wikis further. I'll keep an eye out for when Godex transitions into a native extension; that will definitely warrant more changes to the wikis, good ones nonetheless. As my familiarity increases, I will try to contribute more as I find the time. |
@reeseschultz I've fixed it, not sure why but I needed to add a new line, like this, to make it work:
|
@AndreaCatania Sorry to keep @'ing you—the compilation sections look good, but the SSH and HTTPS sections still need that fix with the newline. Anyway, thanks for incorporating my updates! Edit: Actually, if you think it would help, I'm happy to volunteer as a technical writer for Godex. I can edit docs and wiki modifications before you publish them, just to make sure we catch these issues ahead of time. We can go back-and-forth on things to ensure you get the verbiage you want. This would be good for me to familiarize myself further with Godot and Godex. If you're interested, we can DM on Discord. |
@reeseschultz Hey, apparently I didn't test enough that wiki page, even if I'm pretty sure I did 🤔 In any case, fixed now. If you would like helping with the doc, it would be really helpful so let's continue on godex Discord, I'll be glad to organize things so you can help 🤘 😃 |
Hi. I'm trying to build Godex from the latest Godot master branch
364ea7f280a3f074795e542b16b1d0ec76cf6ce2
.I am using MSVC and building for windows. The error I'm getting is:
Out of curiosity, I tried this setup again from the
28f56e2cbf03a164741f2eade17f9515f887482c
commit. I had the same error.I am able to build Godot from master without Godex installed.
How can I resolve this issue?
Thanks.
The text was updated successfully, but these errors were encountered: