From 4e514a5e0009823a07ad9db0552c7b424b0a6eef Mon Sep 17 00:00:00 2001 From: Brad King Date: Fri, 4 Jan 2008 14:22:57 -0500 Subject: [PATCH] BUG: File installation should overwrite the destination if the file times differ at all rather than only if the source file is newer. Users expect installation to overwrite destination files. This addresses the re-opening of bug#3349. --- Source/cmFileCommand.cxx | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/Source/cmFileCommand.cxx b/Source/cmFileCommand.cxx index 3f93b8d62..618659613 100644 --- a/Source/cmFileCommand.cxx +++ b/Source/cmFileCommand.cxx @@ -949,11 +949,8 @@ bool cmFileInstaller::InstallFile(const char* fromFile, const char* toFile, bool copy = true; if(!always) { - // If both files exist and "fromFile" is not newer than "toFile" - // do not copy. - int timeResult; - if(this->FileTimes.FileTimeCompare(fromFile, toFile, &timeResult) && - timeResult <= 0) + // If both files exist with the same time do not copy. + if(!this->FileTimes.FileTimesDiffer(fromFile, toFile)) { copy = false; }