Improve support for installing with cget #88
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This improves support for installing with cget:
Adds
nlohmann/json
to the requirements.txt so it will automatically be installed with inja. Although, inja can be installed without it(because it can use the embedded version), this will help avoids two version ofnlohmann/json
being installed if two dependencies are using it.Set
BUILD_TESTS
variable based on ctest'sBUILD_TESTING
variable. Cget will set this variable based on whether the user is building for tests or not. So when doingcget install pantor/inja
,BUILD_TESTING
is set toOff
and when doingcget install pantor/inja --test
it will setBUILD_TESTING
toOn
. I kept theBUILD_TESTS
variable so it wont break any backwards compatibility.I added a note in the readme about installing inja with cget. The previous release version of inja is not installable so doing
cget install pantor/inja@v2.0.1
does not work, but it does work from the latest from master. Instead of telling users to install the latest master(iecget install pantor/inja@master
), I added a placeholder<version-tag>
which can be replaced with next released version when it happens.