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

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 '{}' \
             \;