Use platform dependant separator for the PV_PLUGIN_PATH environment variable
This issue was created automatically from an original Mantis Issue. Further discussion may take place here.
The problem is that by some reason Paraview 4.1 only supports semicolon (";") as a separator of the directories for the PV_PLUGIN_PATH variable, though this is natural that separator is platform-dependant (";" on Windows and ":" on Linux). On the other hand ";" is used as a separator of the CMake list items and we encountered with the following problem:
We have a CMake macro which accumulates an environment needed for the correct tests execution:
GENERATE_TESTS_ENVIRONMENT(tests_env) ADD_TEST(TestName TestExecutable) SET_TESTS_PROPERTIES(TestName PROPERTIES ENVIRONMENT "${tests_env}")
The variable tests_env is CMake list which contains a environment variables needed for the test execution:
tests_env is "PATH=path1:path2:path3:...:pathn;LD_LIBRARY_PATH=ldpath1:ldpath2:ldpath3:...:ldpathn;PV_PLUGIN_PATH=pvpath1;pvpath2;pvpath3:...;pvpathn"
The problem is that then CMake parses this list it takes for the PV_PLUGIN_PATH only fist path pvpath1 so far as ";" is a separator for the list items in CMake. Thus some tests fail because Paraview plug-ins located in the pvpath2, pvpath3,...,pvpathn can’t be found.
Attached patch contains a small fix which allows use both ":" and ";" separators on Linux (to avoid possible regressions) and ";" separator on Windows for the PV_PLUGIN_PATH environment variable. Could you please analyze a patch and if possible integrate it the next version of the Paraview?