Browse Source

BUG: Fix required permissions check again

While copying a directory the destination must have owner rwx
permissions.  This corrects our check, this time with correct operator
precedence using parenthesis.
Brad King 16 years ago
parent
commit
03d8b8ad57
1 changed files with 1 additions and 1 deletions
  1. 1 1
      Source/cmFileCommand.cxx

+ 1 - 1
Source/cmFileCommand.cxx

@@ -1214,7 +1214,7 @@ bool cmFileInstaller::InstallDirectory(const char* source,
   // permissions temporarily during file installation.
   // permissions temporarily during file installation.
   mode_t permissions_before = 0;
   mode_t permissions_before = 0;
   mode_t permissions_after = 0;
   mode_t permissions_after = 0;
-  if(permissions & required_permissions == required_permissions)
+  if((permissions & required_permissions) == required_permissions)
     {
     {
     permissions_before = permissions;
     permissions_before = permissions;
     }
     }