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

Include cstdint to fix build on some platforms #90

Merged
merged 1 commit into from
Aug 16, 2023

Conversation

bsaleil
Copy link
Contributor

@bsaleil bsaleil commented Aug 15, 2023

Add cstdint include to fix the build on some platforms. I wasn't able to build LLGL without that include on my Linux machine.

@LukasBanana LukasBanana merged commit 95a6134 into LukasBanana:master Aug 16, 2023
27 checks passed
@LukasBanana
Copy link
Owner

Thanks for the fix. I am curious why this wasn't picked up by the CIS build for Ubuntu. What is your build configuration (GCC or Clang)? Perhaps a different compiler toolchain?

@bsaleil
Copy link
Contributor Author

bsaleil commented Aug 16, 2023

Yeah that's weird. I had the same issue with both gcc 13 and clang 15. Maybe the CIS is using and older version of gcc that was using <cstdint> more widely in its headers ? (https://gcc.gnu.org/gcc-13/porting_to.html)

@LukasBanana
Copy link
Owner

FWIW: The CIS build is using GNU 11.4.0.

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