cryptopp-cmake icon indicating copy to clipboard operation
cryptopp-cmake copied to clipboard

Add cryptopp as a submodule

Open patstew opened this issue 4 years ago • 2 comments

There's a comment in the readme about not being able to add this as a submodule to cryptopp, but why not do it the other way around? It makes it easier to use, e.g. as a submodule in other projects. It also means that each commit in this repo is tied to a specific commit in cryptopp that it can build correctly, so when files are moved around it doesn't cause problems.

patstew avatar Feb 27 '20 17:02 patstew

I assume the problems mentioned are not regarding adding submodule itself, but rather having the files from submodule in the top-level directory. A way around that could be having cryptopp-cmake as a submodule, but then using symlinks in the top level directory.

cryptopp
├─ cryptopp-cmake # a submodule
└─ CMakeLists.txt -> cryptopp-cmake/CMakeLists.txt

janisozaur avatar Mar 08 '21 22:03 janisozaur

I'm still working on the other way around. So you can just checkout the source in the top-level dir, or with starting cmake 3.11, let this be done within the configure-step itself.

Vollstrecker avatar Mar 09 '21 08:03 Vollstrecker

Please check the latest version. We now support many different ways to auto-fetch crypto++ without the additional maintenance overhead of crypto++ as a sub-module.

git submodules require things to be done to keep them up-to-date and consistent which are error-prone.

abdes avatar Aug 22 '22 17:08 abdes