fix cmake header path in install with custom CMAKE_INSTALL_INCLUDEDIR

bebuch commented 8 months ago

Pull request checklist

Read the Contribution Guidelines for detailed information.

[x] Changes are described in the pull request, or an existing issue is referenced.
[x] The test suite compiles and runs without error.
[x] Code coverage is 100%. Test cases can be added by editing the test suite.
[x] The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header files single_include/nlohmann/json.hpp and single_include/nlohmann/json_fwd.hpp. The whole process is described here.

coveralls commented 8 months ago

coverage: 100.0%. remained the same when pulling 4ec42c9918dae5dfa78659c393ae2752ad2905d9 on bebuch:patch-1 into 58d6aa58316a84fc770bdd0409934cdfdf0697dc on nlohmann:develop.

bebuch commented 8 months ago

I don't understand how the CI error is connected to my PR. Please tell me if I need to change something.

nlohmann commented 8 months ago

You're right, this is unrelated.

nlohmann commented 8 months ago

Thanks!

nlohmann / json

fix cmake header path in install with custom CMAKE_INSTALL_INCLUDEDIR #4194

Pull request checklist