Re: [cmake-developers] Problems with external language support when there is a blank in CMAKE_MODULE_PATH

2017-05-25 Thread Chuck Atkins
> > Hi Chuck (off list): > Hi Alan (on list) > but the one above I completely missed. ... > with the hint above I should be able to figure out > all those remaining issues on my own Excellent! Glad to hear it. Inconsistent path quoting is a common pitfal when writing portable CMake code.

Re: [cmake-developers] Problems with external language support when there is a blank in CMAKE_MODULE_PATH

2017-05-25 Thread Chuck Atkins
In addition to that in the Main CMakeLists.txt, make a similar change in CMakeTestAdaCompiler.cmake. Replace: set(CMAKE_MODULE_PATH ${CMAKE_Ada_MODULES}) with: list(INSERT CMAKE_MODULE_PATH 0 \"${CMAKE_Ada_MODULES}\") The additional quoting here is because it's inside a file(WRITE ...) command.

Re: [cmake-developers] Problems with external language support when there is a blank in CMAKE_MODULE_PATH

2017-05-25 Thread Chuck Atkins
Hi Alan, You're missing quotes so when CMake expands CMAKE_SOURCE_DIR then the space in the filename is interpreted as a list separator, no different than: set(my_list foo bar) What you want instead is: set(my_list "foo bar") Change: set(CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/cmake/Modules) To:

Re: [cmake-developers] Problems with external language support when there is a blank in CMAKE_MODULE_PATH

2017-05-22 Thread Alan W. Irwin
So my guess is the measures that have been used so that a blank in the fullpath name works fine for CMake language support files that are installed by CMake have not been extended to the case where CMAKE_MODULE_PATH must be used to find the language support files. I forgot to mention that the

[cmake-developers] Problems with external language support when there is a blank in CMAKE_MODULE_PATH

2017-05-22 Thread Alan W. Irwin
I have been systematically testing PLplot builds for any issues with blanks in the prefix of the source, build, and install trees, and generally all is well under these test conditions. However, we do have two remaining issues which are that our (external) language support for both Ada and D