Browse Source

ENH: Ok, when doing cmake -P you should not have to squish filename next to -P, There should be space between

Andy Cedilnik 22 years ago
parent
commit
51516613e8

+ 2 - 1
Source/cmake.cxx

@@ -217,7 +217,8 @@ bool cmake::SetCacheArgs(const std::vector<std::string>& args)
       }
     else if(arg.find("-P",0) == 0)
       {
-      std::string path = arg.substr(strlen("-P"));
+      i++;
+      std::string path = args[i];
       if ( path.size() == 0 )
         {
         cmSystemTools::Error("No cmake scrpt provided.");

+ 6 - 0
Source/cmakemain.cxx

@@ -185,8 +185,14 @@ int do_cmake(int ac, char** av)
       }
     else if (strncmp(av[i], "-P", strlen("-P")) == 0)
       {
+      if ( i == ac -1 )
+        {
+        cmSystemTools::Error("No script specified for argument -P");
+        }
       script_mode = true;
       args.push_back(av[i]);
+      i++;
+      args.push_back(av[i]);
       }
     else 
       {

+ 1 - 1
Tests/SimpleInstall/CMakeLists.txt

@@ -71,7 +71,7 @@ ADD_CUSTOM_COMMAND(
   POST_BUILD
   COMMAND ${CMAKE_COMMAND}
   ARGS ${EXTRA_INSTALL_FLAGS} 
-  "-P${CMAKE_CURRENT_BINARY_DIR}/cmake_install.cmake"
+  -P "${CMAKE_CURRENT_BINARY_DIR}/cmake_install.cmake"
   COMMENT "Install Project"
   )
 

+ 1 - 1
Tests/SimpleInstallS2/CMakeLists.txt

@@ -71,7 +71,7 @@ ADD_CUSTOM_COMMAND(
   POST_BUILD
   COMMAND ${CMAKE_COMMAND}
   ARGS ${EXTRA_INSTALL_FLAGS} 
-  "-P${CMAKE_CURRENT_BINARY_DIR}/cmake_install.cmake"
+  -P "${CMAKE_CURRENT_BINARY_DIR}/cmake_install.cmake"
   COMMENT "Install Project"
   )