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, 2 insertions, 10 deletions

diff --git a/mkinitcpio b/mkinitcpio
index ad96a4f..42b8290 100755
--- a/mkinitcpio
+++ b/mkinitcpio
@@ -125,8 +125,7 @@ hook_help() {
         return 1
     fi
 
-    if [[ -L $script ]]; then
-        resolved=$(readlink -e "$script")
+    if resolved=$(readlink "$script") && [[ ${script##*/} != "${resolved##*/}" ]]; then
         msg "This hook is deprecated. See the '%s' hook" "${resolved##*/}"
         return 0
     fi
@@ -155,14 +154,7 @@ hook_list() {
             [[ -e $hook || -L $hook ]] || continue
 
             # handle deprecated hooks and point to replacement
-            if [[ -L $hook ]]; then
-                resolved=$(readlink -e "$hook")
-
-                if [[ -z $resolved ]]; then
-                    error "found broken symlink '%s'" "$hook"
-                    continue
-                fi
-
+            if resolved=$(readlink "$hook") && [[ ${hook##*/} != "${resolved##*/}" ]]; then
                 resolved=${resolved##*/}
 
                 if ! index_of "$resolved" "${depr[@]}"; then