intel / isa-l_crypto

Other
271 stars 80 forks source link

Makefile.nmake only builds Release configuration, resulting in mixed CRT warning in VS #120

Open gh-andre opened 1 year ago

gh-andre commented 1 year ago

Visual C++ expects different MSVC CRT options for Debug and Release builds and current Makefile.nmake builds only libraries for Release configurations. I realize that it's generated from tools/gen_nmake.mk and I didn't look into how it is plugged in for a PR, but if you are interested in a patch, in general, here's one for Makefile.nmake:

https://github.com/StoneStepsInc/isa-lib-crypto-nuget/blob/master/patches/01-nmake-debug-release.patch

With this patch, one can build Debug/Release libraries like this:

nmake -f Makefile.nmake CONFIG=DEBUG
nmake -f Makefile.nmake CONFIG=RELEASE