-
Notifications
You must be signed in to change notification settings - Fork 479
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
Switch to a CMake-based build #106
Comments
Caveat: Doesn't support the `--coverage` linking option, used in the original `Makefile`, explicitly
Caveat: Doesn't support the `--coverage` linking option, used in the original `Makefile`, explicitly
Caveat: Doesn't support the `--coverage` linking option, used in the original `Makefile`, explicitly
Caveat: Doesn't support the `--coverage` linking option, used in the original `Makefile`, explicitly
This project is intended to be included in other projects, it is not build as library. Using CMake for unittest is fine, but compilation options may cause problems when other 'outer' build system is used ... |
@ledvinap : CMake is even better for use in other projects, and I've included a mechanism for installing and exporting. Also, you can always create a Makefile using CMake, if you really like Makefiles for some reason. And Makefile's suffer from the same problem, even worse: For example, Windows people don't use Makefile, while with CMake you can generate MSVS projects, or nmake scripts, or whatever you like. Same goes for other platforms. Also, if someone wants to just use the two source files ( But - if there's a usage scenario I've missed, please elaborate... |
It should be fine as long as print.[ch] is not dependent on CMake use .. |
…sured all relevant numeric fields actually get numbers. Removed some commented-out code.
…sured all relevant numeric fields actually get numbers. Removed some commented-out code.
Instead of a Makefile which makes several assumptions it doesn't need to, consider basing the build on the CMake build system generator. This will allow:
The text was updated successfully, but these errors were encountered: