godot icon indicating copy to clipboard operation
godot copied to clipboard

Missing headers when building

Open SeleDreams opened this issue 3 years ago • 7 comments

Since I was interested by the concept of ECS in godot I wanted to try it and build it but i got missing headers when building it I'm building it for windows using WSL image

SeleDreams avatar Oct 07 '20 22:10 SeleDreams

Interesting, I'll try that soon, thanks for the report.

AndreaCatania avatar Oct 08 '20 11:10 AndreaCatania

@SeleDreams can you please pass me the command line you used to compile it? So I can reproduce this exact use case.

AndreaCatania avatar Oct 10 '20 08:10 AndreaCatania

scons platform=windows use_mingw=yes

Le sam. 10 oct. 2020 à 10:24, Andrea Catania [email protected] a écrit :

@SeleDreams https://github.com/SeleDreams can you please pass me the command line you used to compile it? So I can reproduce this exact use case.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AndreaCatania/godot/issues/12#issuecomment-706511042, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD4UF4MVKRL6OKSJGEI2CGLSKAK3FANCNFSM4SH7NITQ .

SeleDreams avatar Oct 10 '20 09:10 SeleDreams

The issue was much simpler than I was looking for: indeed the directory storages was ignored by the rule https://github.com/AndreaCatania/godot/blob/master/.gitignore#L40 so the directory was not present into the project.

I could have easily lost all the code, without this issue, so thank a lot for opening this.

By the way, it's now fixed, you just need to do a pull:

git fetch my_repository_name 
git reset --hard my_repository_name ecs // This command is necessary because I've rebased to the newest Godot version.

AndreaCatania avatar Oct 10 '20 11:10 AndreaCatania

Please close the issue if it's done, or just tell me if anything is still not working.

AndreaCatania avatar Oct 10 '20 11:10 AndreaCatania

image Seems like I'm still getting different errors this time

SeleDreams avatar Oct 11 '20 14:10 SeleDreams

Fixed with: https://github.com/AndreaCatania/godot/commit/3e62bf4b75b72ef2f6663f3457e6f566bef0073b. Before I was trying to solve the warning without adding additional memory allocation, but seems it's not possible to have a cross compiler solution, so I just added a static phantom data.

AndreaCatania avatar Oct 11 '20 18:10 AndreaCatania