Be more conservative when setting write access.

Only need to set write access on files that are currently read-only.
Cuts down considerably on debugging output.
This commit is contained in:
Oliver Hamlet
2015-07-14 18:52:23 +01:00
parent 976aaa624a
commit b2cbc3fa2d
+4 -2
View File
@@ -128,8 +128,10 @@ namespace loot {
void GitHelper::FixRepoPermissions(const boost::filesystem::path& path) {
BOOST_LOG_TRIVIAL(trace) << "Recursively setting write permission on directory: " << path;
for (fs::recursive_directory_iterator it(path); it != fs::recursive_directory_iterator(); ++it) {
BOOST_LOG_TRIVIAL(trace) << "Setting write permission for: " << it->path();
fs::permissions(it->path(), fs::add_perms | fs::owner_write | fs::group_write | fs::others_write);
if ((it->status().permissions() & (fs::owner_write | fs::group_write | fs::others_write)) == 0) {
BOOST_LOG_TRIVIAL(trace) << "Setting write permission for: " << it->path();
fs::permissions(it->path(), fs::add_perms | fs::owner_write);
}
}
}