-
Notifications
You must be signed in to change notification settings - Fork 70
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
CMake config file #41
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we also add test and coverage targets? That would be necessary if we want to replace the GNU Makefiles.
I don't know how to do that. Can you continue the work? |
@ssrlive commented on the current CMake support status in #48; I am moving this here since there is no official CMake support in Here is the comment: test it on Windows / Linux / macOS, all the Release buildings are failed.
|
Also @ssrlive , I can't provide CMake support. The GNU make solution works fine. I am happy to merge a working If that continues to be an issue for you, I am also happy to open a ticket that asks for CMake support and to tag it with a help-wanted tag. |
Couldn't you provide this CMakeLists.txt besides the regular Makefile? There's no harm in having both, even though CMake can generate a Makefile, you usually don't do this within the project directory. Having CMake support would make it much easier to integrate gc into other projects e.g. using git submodules. |
@mupfdev, misunderstanding: as mentioned in the previous comment I'd be happy to merge a |
CMake configure file ready.