cryptopp-cmake icon indicating copy to clipboard operation
cryptopp-cmake copied to clipboard
verified publisher icon noloader

Fixed missing `-D` in several compile definitions

Open codeshaunted opened this issue 3 years ago • 1 comments

Several compile definitions in CRYPTOPP_COMPILE_DEFINITIONS weren't prefixed with -D, which caused compilation errors for me on Windows with CMake 2.21. I've gone through and added the prefix to every one I could find.

codeshaunted avatar May 16 '22 01:05 codeshaunted

Any chance someone can look into merging this? My projects won't build on Windows without this change, and I don't want to be using a custom fork for my submodules.

codeshaunted avatar Jun 23 '22 04:06 codeshaunted

CMake normally adds the -D automatically to compiler definitions: https://cmake.org/cmake/help/latest/prop_sf/COMPILE_DEFINITIONS.html

The COMPILE_DEFINITIONS property may be set to a semicolon-separated list of preprocessor definitions using the syntax VAR or VAR=value.

Your version of cmake is not even supported anymore. Please use a more recent version.

abdes avatar Aug 22 '22 17:08 abdes

I think that may have been a typo and meant to be 3.21. Will later reopen if that’s the case.

codeshaunted avatar Aug 22 '22 18:08 codeshaunted

Yes, please check with the current master and if you still have a problem, please reopen and update to the latest master, I will check it again.

abdes avatar Aug 22 '22 18:08 abdes