Skip to content
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

Game SDK claims 'std' does not name a type (Godot 3.2.4 beta 4) #20

Closed
jordigcs opened this issue Jan 1, 2021 · 5 comments · Fixed by #24 or #28
Closed

Game SDK claims 'std' does not name a type (Godot 3.2.4 beta 4) #20

jordigcs opened this issue Jan 1, 2021 · 5 comments · Fixed by #24 or #28

Comments

@jordigcs
Copy link
Contributor

jordigcs commented Jan 1, 2021

I'm experiencing errors when trying to compile Godot 3.2.4 with Godotcord on Ubuntu 20.10
image
This is with a fresh clone of both Godot and Godotcord, after running setup.sh
I built with scons -j6 platform=x11 target=release_debug use_lto=yes tools=yes
Could this be an issue with Godotcord or with the Game SDK?

Does this compile on Linux for you @Drachenfrucht1?

Specs:
Godot 3.2.4 beta 4
Ubuntu 20.10

@jordigcs jordigcs changed the title Game SDK CPP claims 'std' does not name a type Game SDK claims 'std' does not name a type (Godot 3.2.4 beta 4) Jan 1, 2021
@jordigcs
Copy link
Contributor Author

jordigcs commented Jan 1, 2021

Reproducible on 3.2.3 as well

@jordigcs
Copy link
Contributor Author

jordigcs commented Jan 1, 2021

After removing all parts of the GameSDK that use std:: (for example std::int32_t becomes int32_t) and adding using namespace std; to types.h, the engine compiles successfully. But when I try to run an empty scene in an empty project, I get the following error:
image
Even though the file structure is correct:
image

Any clue why this could be happening?

Moving this to a separate issue (#22)

@Drachenfrucht1
Copy link
Owner

This is a problem with the header files provided by discord. The reason could be that only stdint.h is included. A quick google search said that only cstdint provides the fixed width integer types in the std namespace.

An alternative to your fix is to include the cstdint header in the types.h file.
We could try to add a fix to the setup scripts as long as discord hasn't fixed this.

@jordigcs
Copy link
Contributor Author

jordigcs commented Jan 1, 2021

This is a problem with the header files provided by discord. The reason could be that only stdint.h is included. A quick google search said that only cstdint provides the fixed width integer types in the std namespace.

An alternative to your fix is to include the cstdint header in the types.h file.
We could try to add a fix to the setup scripts as long as discord hasn't fixed this.

I'll try to add it to the setup scripts later today, I also confirmed the issue also happens on Windows, so I'll try to add it to the .bat file as well.

@jordigcs
Copy link
Contributor Author

jordigcs commented Jan 1, 2021

@Drachenfrucht1 I fixed this issue in setup.sh (PR #24) but couldn't figure out how to do it in setup.bat since I'm not experienced with Batch files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants