[cmake-commits] hoffman committed cmLocalGenerator.cxx 1.185 1.186

[cmake-commits at cmake.org]

Update of /cvsroot/CMake/CMake/Source
In directory public:/mounts/ram/cvs-serv9290

Modified Files:
	cmLocalGenerator.cxx 
Log Message:
ENH: try and use \ for space and () escapes


Index: cmLocalGenerator.cxx
===================================================================
RCS file: /cvsroot/CMake/CMake/Source/cmLocalGenerator.cxx,v
retrieving revision 1.185
retrieving revision 1.186
diff -u -d -r1.185 -r1.186
--- cmLocalGenerator.cxx	24 Feb 2007 01:37:35 -0000	1.185
+++ cmLocalGenerator.cxx	25 Feb 2007 21:13:33 -0000	1.186
@@ -2066,6 +2066,21 @@
     }
   if( output == SHELL)
     {
+    // for shell commands if force unix is on, but this->WindowsShell
+    // is true, then turn off force unix paths for the output path
+    // so that the path is windows style and will work with windows
+    // cmd.exe.
+    bool forceOn =  cmSystemTools::GetForceUnixPaths();
+    if(forceOn && this->WindowsShell)
+      {
+      cmSystemTools::SetForceUnixPaths(false);
+      }
+    result = cmSystemTools::ConvertToOutputPath(result.c_str());
+    if(forceOn && this->WindowsShell)
+      {
+      cmSystemTools::SetForceUnixPaths(true);
+      }
+
     // For the MSYS shell convert drive letters to posix paths, so
     // that c:/some/path becomes /c/some/path.  This is needed to
     // avoid problems with the shell path translation.
@@ -2077,20 +2092,19 @@
         result[0] = '/';
         }
       }
-    if(this->WindowsShell)
+    // if this is unix then we need to escape () in the shell
+#if !defined(WIN32) || defined(CYGWIN)
+    forceOn = true;
+#endif
+    if(forceOn )
       {
-      std::string::size_type pos = 0;
-      while((pos = result.find('/', pos)) != std::string::npos)
-        {
-        result[pos] = '\\';
-        pos++;
-        }
+      result = cmSystemTools::EscapeForUnixShell(result);
       }
-    result = this->EscapeForShell(result.c_str(), true, false);
     }
   return result;
 }
 
+
 //----------------------------------------------------------------------------
 void
 cmLocalGenerator