Add support for linux platform #10
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add CMakeLists.txt for project generation. Fix errors and missing includes in GCC/clang. Add support for external libraries (via cmake FetchContent). Note that GLFW needs a newer (but compatible) version for newer cmake to work correctly. Add platform code for GetTimeMicroseconds and to get the current working directory for file io. Add .gitignore to ignore directories that are generally created by cmake/clangd
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello. I was going to read and follow your book on linux, but I read there was no official support for any platform but Windows.
I went ahead and added support for linux (via cmake and fetchcontent) and cleaned up some of the warnings and errors that were present on compilers other than MSVC.
I am not sure if you'd be interested at all in this change or supporting non-windows platform at all.
I understand if you are not, but in case I thought someone looking for linux support might stumble on this PR and could use it as a starting point.
Thank you for the code and the book :)