refactor(compatdir): resolve paths in a function

Author: akinomyoga

bash_completion

@@ -3303,47 +3303,65 @@ else
     }
 fi
 
-# source compat completion directory definitions
-_comp__init_compat_dirs=()
-if [[ ${BASH_COMPLETION_COMPAT_DIR-} ]]; then
-    _comp__init_compat_dirs+=("$BASH_COMPLETION_COMPAT_DIR")
-else
-    _comp__init_compat_dirs+=(/etc/bash_completion.d)
-    # Similarly as for the "completions" dir, look up from relative to
-    # bash_completion, primarily for installed-with-prefix and
-    # run-in-place-from-git-clone setups.  Notably we do it after the system
-    # location here, in order to prefer in-tree variables and functions.
-    if [[ ${BASH_SOURCE%/*} == */share/bash-completion ]]; then
-        _comp__init_compat_dir=${BASH_SOURCE%/share/bash-completion/*}/etc/bash_completion.d
-    elif [[ $BASH_SOURCE == */* ]]; then
-        _comp__init_compat_dir="${BASH_SOURCE%/*}/bash_completion.d"
+# List custom/extra completion files to source on the startup
+## @param $1 path Path to "bash_completion"
+## @var[out] _comp__init_startup_configs
+_comp__init_collect_startup_configs()
+{
+    local base_path=${1:-${BASH_SOURCE[1]}}
+    _comp__init_startup_configs=()
+
+    # source compat completion directory definitions
+    local -a compat_dirs=()
+    local compat_dir
+    if [[ ${BASH_COMPLETION_COMPAT_DIR-} ]]; then
+        compat_dirs+=("$BASH_COMPLETION_COMPAT_DIR")
     else
-        _comp__init_compat_dir=./bash_completion.d
-    fi
-    [[ ${_comp__init_compat_dirs[0]} == "$_comp__init_compat_dir" ]] ||
-        _comp__init_compat_dirs+=("$_comp__init_compat_dir")
-fi
-for _comp__init_compat_dir in "${_comp__init_compat_dirs[@]}"; do
-    [[ -d $_comp__init_compat_dir && -r $_comp__init_compat_dir && -x $_comp__init_compat_dir ]] || continue
-    _comp_expand_glob _comp__init_files '"$_comp__init_compat_dir"/*'
-    # shellcheck disable=SC2154
-    for _comp__init_file in "${_comp__init_files[@]}"; do
-        [[ ${_comp__init_file##*/} != @($_comp_backup_glob|Makefile*|${BASH_COMPLETION_COMPAT_IGNORE-}) &&
-            -f $_comp__init_file && -r $_comp__init_file ]] && . "$_comp__init_file"
+        compat_dirs+=(/etc/bash_completion.d)
+        # Similarly as for the "completions" dir, look up from relative to
+        # bash_completion, primarily for installed-with-prefix and
+        # run-in-place-from-git-clone setups.  Notably we do it after the
+        # system location here, in order to prefer in-tree variables and
+        # functions.
+        if [[ ${base_path%/*} == */share/bash-completion ]]; then
+            compat_dir=${base_path%/share/bash-completion/*}/etc/bash_completion.d
+        elif [[ $base_path == */* ]]; then
+            compat_dir="${base_path%/*}/bash_completion.d"
+        else
+            compat_dir=./bash_completion.d
+        fi
+        [[ ${compat_dirs[0]} == "$compat_dir" ]] ||
+            compat_dirs+=("$compat_dir")
+    fi
+    for compat_dir in "${compat_dirs[@]}"; do
+        [[ -d $compat_dir && -r $compat_dir && -x $compat_dir ]] || continue
+        local compat_files
+        _comp_expand_glob compat_files '"$compat_dir"/*'
+        local compat_file
+        for compat_file in "${compat_files[@]}"; do
+            [[ ${compat_file##*/} != @($_comp_backup_glob|Makefile*|${BASH_COMPLETION_COMPAT_IGNORE-}) &&
+                -f $compat_file && -r $compat_file ]] &&
+                _comp__init_startup_configs+=("$compat_file")
+        done
     done
-done
-unset -v _comp__init_compat_dirs _comp__init_compat_dir _comp__init_files _comp__init_file
 
-# source user completion file
-#
-# Remark: We explicitly check that $user_completion is not '/dev/null' since
-#   /dev/null may be a regular file in broken systems and can contain arbitrary
-#   garbages of suppressed command outputs.
-_comp__init_user_file=${BASH_COMPLETION_USER_FILE:-~/.bash_completion}
-[[ $_comp__init_user_file != "${BASH_SOURCE[0]}" && $_comp__init_user_file != /dev/null && -r $_comp__init_user_file && -f $_comp__init_user_file ]] &&
-    . "$_comp__init_user_file"
-unset -v _comp__init_user_file
+    # source user completion file
+    #
+    # Remark: We explicitly check that $user_completion is not '/dev/null'
+    #   since /dev/null may be a regular file in broken systems and can contain
+    #   arbitrary garbages of suppressed command outputs.
+    local user_file=${BASH_COMPLETION_USER_FILE:-~/.bash_completion}
+    [[ $user_file != "$base_path" && $user_file != /dev/null && -r $user_file && -f $user_file ]] &&
+        _comp__init_startup_configs+=("$user_file")
 
+    unset -f "$FUNCNAME"
+}
+_comp__init_collect_startup_configs "$BASH_SOURCE"
+# shellcheck disable=SC2154
+for _comp_init_startup_config in "${_comp__init_startup_configs[@]}"; do
+    . "$_comp_init_startup_config"
+done
+unset -v _comp__init_startup_configs _comp_init_startup_config
 unset -f have
 unset -v have