7x31/shellcheck
1
0
Fork 0
mirror of https://github.com/koalaman/shellcheck.git synced 2025-10-03 19:29:44 +08:00
Code Issues Packages Projects Releases Wiki Activity

Updated SC2174 (markdown)

koalaman
2016-01-09 16:47:17 -08:00
parent a04792e7ae
commit a1deac9de2
1 changed files with 14 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
SC2174.md

@@ -1,4 +1,4 @@
## mkdir -pm ignores -m for parent directories
## When used with -p, -m only applies to the deepest directory.
### Problematic code:
@@ -6,8 +6,19 @@
mkdir -p -m 0755 foo/bar/baz
```
### Correct code:
```sh
mkdir -p foo/bar/baz
chmod 0755 foo/bar/baz foo/bar foo
```
### Rationale:
When using `-m 0755`, the mode of the directory created will be set to 0755. When using `-p`, parent directories which do not exist will be created, but the mode specified by `-m` will only be used on the non-parent directory. The parent directories will get their access mode the default way, via [umask(2)].
When using `-m 0755`, the mode of the directory created will be set to 0755. When using `-p`, parent directories which do not exist will be created, but the mode specified by `-m` will only be used on the last directory. The parent directories will get their access mode the default way, via [umask(2)].
[umask(2)]: http://man7.org/linux/man-pages/man2/umask.2.html
[umask(2)]: http://man7.org/linux/man-pages/man2/umask.2.html
### Exceptions:
ShellCheck does not warn if the path only has one component, as in `mkdir -p -m 0755 mydir`, but will not attempt to determine whether this applies for a variable as in `mkdir -p -m 0755 "$mydir"`. You can mkdir/chmod separately or [[ignore]] this message.