diff options
author | Mike Burns <mike@mike-burns.com> | 2015-11-13 13:27:56 +0100 |
---|---|---|
committer | Mike Burns <mike@mike-burns.com> | 2015-11-13 13:34:52 +0100 |
commit | a801c4c58902cae7b835d890487edc39bd7da142 (patch) | |
tree | c05540025b1ce37ce10a00eded95ef6f318f299b /share | |
parent | 21ba58a06d82bfaec8e2daf67a694311e3cc3034 (diff) |
Use $LOGNAME instead of $USER
The $USER environment variable is not guaranteed to exist, but $LOGNAME
is defined by POSIX[0] (Environment Variables, section 8.3 Other
Environment Variables). Use that instead.
Thanks to Scott Stevenson for pointing out this solution, and Debian for
raising the problem.
[0]: http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap08.html#tag_08_03
Diffstat (limited to 'share')
-rw-r--r-- | share/rcm.sh.in | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/share/rcm.sh.in b/share/rcm.sh.in index c7a3dfb..ed5d6f9 100644 --- a/share/rcm.sh.in +++ b/share/rcm.sh.in @@ -130,7 +130,7 @@ run_hooks() { # The former is however a bit better when it comes to security. On the other hand # running these hooks imply some level of trust; surely one doesn't clone somebody # else's dotfiles repository without reviewing the hooks before doing an `rcup`? - find "$hook_file" -type f \( \( -user $USER -perm -100 \) -o -perm -001 \) \ + find "$hook_file" -type f \( \( -user $LOGNAME -perm -100 \) -o -perm -001 \) \ -exec \ sh -c 'cd "`dirname $1`" && ./"`basename $1`"' arg0 '{}' \ \; |