diff options
author | Brad King <brad.king@kitware.com> | 2017-01-16 19:06:29 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2017-01-16 19:06:29 (GMT) |
commit | ee77941a1fdf3a617207b00a7f2e9315eaef7b31 (patch) | |
tree | 9fa7da0722af50bd783b33961e931c2b687b1d82 /Source/kwsys/SystemTools.cxx | |
parent | 35ad12f99e71515f9cc65b1219160b88095a5f6b (diff) | |
parent | 219c7bdcb16cd1dfe2e72befa4334c5b8147e1d7 (diff) | |
download | CMake-ee77941a1fdf3a617207b00a7f2e9315eaef7b31.zip CMake-ee77941a1fdf3a617207b00a7f2e9315eaef7b31.tar.gz CMake-ee77941a1fdf3a617207b00a7f2e9315eaef7b31.tar.bz2 |
Merge branch 'upstream-KWSys' into update-kwsys
* upstream-KWSys:
KWSys 2017-01-16 (a423d829)
Diffstat (limited to 'Source/kwsys/SystemTools.cxx')
-rw-r--r-- | Source/kwsys/SystemTools.cxx | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/Source/kwsys/SystemTools.cxx b/Source/kwsys/SystemTools.cxx index b018a43..97dd4ae 100644 --- a/Source/kwsys/SystemTools.cxx +++ b/Source/kwsys/SystemTools.cxx @@ -763,7 +763,7 @@ bool SystemTools::MakeDirectory(const char* path) bool SystemTools::MakeDirectory(const std::string& path) { - if (SystemTools::FileExists(path)) { + if (SystemTools::PathExists(path)) { return SystemTools::FileIsDirectory(path); } if (path.empty()) { @@ -1280,7 +1280,7 @@ bool SystemTools::PathCygwinToWin32(const char* path, char* win32_path) bool SystemTools::Touch(const std::string& filename, bool create) { - if (!SystemTools::FileExists(filename)) { + if (!SystemTools::PathExists(filename)) { if (create) { FILE* file = Fopen(filename, "a+b"); if (file) { @@ -4289,7 +4289,7 @@ bool SystemTools::GetLineFromStream(std::istream& is, std::string& line, // if we read too much then truncate the buffer if (leftToRead > 0) { if (static_cast<long>(length) > leftToRead) { - buffer[leftToRead - 1] = 0; + buffer[leftToRead] = 0; leftToRead = 0; } else { leftToRead -= static_cast<long>(length); @@ -4389,10 +4389,7 @@ bool SystemTools::SetPermissions(const char* file, mode_t mode, bool SystemTools::SetPermissions(const std::string& file, mode_t mode, bool honor_umask) { - // TEMPORARY / TODO: After FileExists calls lstat() instead of - // access(), change this call to FileExists instead of - // TestFileAccess so that we don't follow symlinks. - if (!SystemTools::TestFileAccess(file, TEST_FILE_OK)) { + if (!SystemTools::PathExists(file)) { return false; } if (honor_umask) { |