Conversation
| FetchContent_Declare( | ||
| Boost | ||
| GIT_REPOSITORY "https://github.com/boostorg/boost.git" | ||
| GIT_TAG "boost-1.69.0" |
There was a problem hiding this comment.
Would be nice to have the git version specified in 1 place, and string() code set up to format it as needed in the URL and Git repo cases. This way when you upgrade to a new version of Boost, you aren't changing 3-4+ places
There was a problem hiding this comment.
Done. Note that you can always use FetchContent_Declare in the global CMake project before using add_subdirectory() on boost-cmake to chose the branch or tag you prefer.
Still, I moved those strings to option() as it can be more convenient.
There was a problem hiding this comment.
I apologize for the confusion, I didn't mean a configuration setting. I meant just for maintainability, a "local" variable to store it to make it easier for the maintainers to update the version later. In the case where the user wants to override the version, they must also specify a tag or SHA1, so 1 variable doesn't quite cut it. In that case, yeah I agree, they need to use Declare.
Orphis
force-pushed
the
git_support
branch
3 times, most recently
from
516b4fe to
e29f0a6
Compare
2 participants
No description provided.