install: compat fix for mode of created files.
authorDenys Vlasenko <vda.linux@googlemail.com>
Mon, 15 Jun 2009 16:13:51 +0000 (18:13 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Mon, 15 Jun 2009 16:13:51 +0000 (18:13 +0200)
By Mat Hostetter (mhostetter AT tilera.com)

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
coreutils/install.c

index 24fb402af5213c03c2df8694c0108f27e79cf966..2e604bec7837755e2ac0c365e48fb4eee653d95a 100644 (file)
@@ -129,7 +129,7 @@ int install_main(int argc, char **argv)
        if (opts & OPT_PRESERVE_TIME) {
                copy_flags |= FILEUTILS_PRESERVE_STATUS;
        }
-       mode = 0666;
+       mode = 0755; /* GNU coreutils 6.10 compat */
        if (opts & OPT_MODE)
                bb_parse_mode(mode_str, &mode);
        uid = (opts & OPT_OWNER) ? get_ug_id(uid_str, xuname2uid) : getuid();
@@ -175,8 +175,9 @@ int install_main(int argc, char **argv)
                        }
                }
 
-               /* Set the file mode */
-               if ((opts & OPT_MODE) && chmod(dest, mode) == -1) {
+               /* Set the file mode (always, not only with -m).
+                * GNU coreutils 6.10 is not affected by umask. */
+               if (chmod(dest, mode) == -1) {
                        bb_perror_msg("can't change %s of %s", "permissions", dest);
                        ret = EXIT_FAILURE;
                }