On Wednesday 24 August 2011 18:40:32 Michael Jansen wrote:
> Please consider to not use the lower-case variant. Instead
> use QJson-config.cmake or anything else with at least one UpperCase Letter

So I just have to rename qjson-config.cmake to QJson-config.cmake, right?

> Please change this in the root CMakeLists.txt (85++):
> 
> INSTALL(FILES ${CMAKE_CURRENT_BINARY_DIR}/qjson-config.cmake
>               ${CMAKE_CURRENT_BINARY_DIR}/qjson-config-version.cmake
>         DESTINATION "lib/cmake/qjson")
> 
> to
> 
> INSTALL(FILES ${CMAKE_CURRENT_BINARY_DIR}/qjson-config.cmake
>               ${CMAKE_CURRENT_BINARY_DIR}/qjson-config-version.cmake
>         DESTINATION "${CMAKECONFIG_INSTALL_DIR}")

I'm ready to push this change to master.

Is there anything else I should change?

Cheers
Flavio

------------------------------------------------------------------------------
EMC VNX: the world's simplest storage, starting under $10K
The only unified storage solution that offers unified management 
Up to 160% more powerful than alternatives and 25% more efficient. 
Guaranteed. http://p.sf.net/sfu/emc-vnx-dev2dev
_______________________________________________
QJson-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qjson-devel

Reply via email to