From d9e060e0625edfe523210f66e023b3367f5a2775 Mon Sep 17 00:00:00 2001 From: Brad King Date: Tue, 30 May 2006 11:15:27 -0400 Subject: [PATCH] BUG: Fixed cmake -E remove return code. --- Source/cmake.cxx | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/Source/cmake.cxx b/Source/cmake.cxx index 08677017f..6bbf9d116 100644 --- a/Source/cmake.cxx +++ b/Source/cmake.cxx @@ -899,15 +899,22 @@ int cmake::ExecuteCMakeCommand(std::vector& args) // Remove file else if (args[1] == "remove" && args.size() > 2) { + bool force = false; for (std::string::size_type cc = 2; cc < args.size(); cc ++) { - if(args[cc] != "-f") + if(args[cc] == "\\-f" || args[cc] == "-f") { - if(args[cc] == "\\-f") + force = true; + } + else + { + // Complain if the file could not be removed, still exists, + // and the -f option was not given. + if(!cmSystemTools::RemoveFile(args[cc].c_str()) && !force && + cmSystemTools::FileExists(args[cc].c_str())) { - args[cc] = "-f"; + return 1; } - cmSystemTools::RemoveFile(args[cc].c_str()); } } return 0;