From 512d5dd1e3afb26302e5fa231a8b17f6274df528 Mon Sep 17 00:00:00 2001 From: Bill Hoffman Date: Sun, 25 Feb 2007 16:13:33 -0500 Subject: [PATCH] ENH: try and use \ for space and () escapes --- Source/cmLocalGenerator.cxx | 30 ++++++++++++++++++++++-------- 1 file changed, 22 insertions(+), 8 deletions(-) diff --git a/Source/cmLocalGenerator.cxx b/Source/cmLocalGenerator.cxx index 834fe10d4..953a4f4a5 100644 --- a/Source/cmLocalGenerator.cxx +++ b/Source/cmLocalGenerator.cxx @@ -2066,6 +2066,21 @@ std::string cmLocalGenerator::Convert(const char* source, } 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 @@ std::string cmLocalGenerator::Convert(const char* source, 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