Avoid erroneous deprecation warnings

Consistently handle hooks as symlinks by relying on the exit status of
readlink instead of an lstat and always, additionally, examine the hooks
basenames to ensure that we aren't warning about foo deprecating foo
(because the hook is a symlink to somewhere else that the hook is
actually stored on disk.

ref: https://bugs.archlinux.org/task/55323

1 files changed, 4 insertions, 5 deletions

diff --git a/functions b/functions
index 941312f..53557e7 100644
--- a/functions
+++ b/functions
@@ -695,7 +695,7 @@ initialize_buildroot() {
 }
 
 run_build_hook() {
-    local hook=$1 script= realscript=
+    local hook=$1 script= resolved=
     local MODULES=() BINARIES=() FILES=() SCRIPT=
 
     # find script in install dirs
@@ -705,11 +705,10 @@ run_build_hook() {
     fi
 
     # check for deprecation
-    if [[ -L $script ]]; then
-        realscript=$(readlink -e "$script")
+    if resolved=$(readlink "$script") && [[ ${script##*/} != "${resolved##*/}" ]]; then
         warning "Hook '%s' is deprecated. Replace it with '%s' in your config" \
-            "${script##*/}" "${realscript##*/}"
-        script=$realscript
+            "${script##*/}" "${resolved##*/}"
+        script=$resolved
     fi
 
     # source