Shellcheck suggestions

Joel Grunbaum

2021-12-24 d2c44c5903537b478900c30c8255fc9d64549b6e

Shellcheck suggestions

 INSTALL.sh

@@ -3,12 +3,12 @@
install_links() {
    . ./DICT
    if [ -n "$FILES" ]; then
    for i in `seq 1 $(echo "$FILES" | wc -w)`; do
        file=$(echo "$FILES" | cut -d' ' -f $i)
        loc=$(echo "$LOCATIONS" | cut -d' ' -f $i)
        while [ 1 ]; do
    for i in $(seq 1 "$(echo "$FILES" | wc -w)"); do
        file=$(echo "$FILES" | cut -d' ' -f "$i")
        loc=$(echo "$LOCATIONS" | cut -d' ' -f "$i")
        while true; do
            CHOICE="overwrite"
            if [ -a "$loc" ] && [ "$FORCE" = "0" ]; then
            if [ -e "$loc" ] && [ "$FORCE" = "0" ]; then
                echo "WARNING: \"$loc\" exists, (overwrite, change, nothing): "
                read CHOICE
            fi
@@ -19,10 +19,10 @@
                    ln -sf "$(pwd)/$file" "$loc"; break;;
                c|change)
                    tmpfile=$(mktemp)
                    echo "$loc" >> $tmpfile
                    echo "$loc" >> "$tmpfile"
                    $EDITOR "$tmpfile"
                    loc=$(cat $tmpfile)
                    rm $tmpfile
                    loc=$(cat "$tmpfile")
                    rm "$tmpfile"
                    unset tmpfile;;
                n|nothing)
                    break;;
@@ -34,62 +34,62 @@
    done
    fi
    if [ -n "$SUDO_FILES" ]; then
    for i in `seq 1 $(echo "$SUDO_FILES" | wc -w)`; do
        file=$(echo "$SUDO_FILES" | cut -d' ' -f $i)
        loc=$(echo "$SUDO_LOCATIONS" | cut -d' ' -f $i)
    for i in $(seq 1 "$(echo "$SUDO_FILES" | wc -w)"); do
        file=$(echo "$SUDO_FILES" | cut -d' ' -f "$i")
        loc=$(echo "$SUDO_LOCATIONS" | cut -d' ' -f "$i")
        dir=$(echo "$loc" | rev | cut -d'/' -f2- | rev)
        sudo mkdir -p "$dir"
        sudo ln -sf "$(pwd)/$file" "$loc"
    done
    fi
    if [ "$(type -t custom)" = "function" ]; then
    custom install
    if type custom | grep -q "function" ; then
        custom install
    fi
}

remove_links() {
    . ./DICT
    if [ -n "$FILES" ]; then
    for i in `seq 1 $(echo $FILES | wc -w)`; do
        loc=$(echo "$LOCATIONS" | cut -d' ' -f $i)
    for i in $(seq 1 "$(echo "$FILES" | wc -w)"); do
        loc=$(echo "$LOCATIONS" | cut -d' ' -f "$i")
        rm "$loc" 2>/dev/null
        loc=$(echo "$loc" | rev | cut -d'/' -f2- | rev)
        while [ -z "$(ls -A $loc)" ]; do
        loc=$(dirname "$loc")
        while [ -z "$(ls -A "$loc")" ]; do
        rm -r "$loc"
        loc=$(echo "$loc" | rev | cut -d'/' -f2- | rev)
        loc=$(dirname "$loc")
        done 2>/dev/null
    done
    fi
    if [ -n "$SUDO_FILES" ]; then
    for i in `seq 1 $(echo $SUDO_FILES | wc -w)`; do
        loc=$(echo "$SUDO_LOCATIONS" | cut -d' ' -f $i)
    for i in $(seq 1 "$(echo "$SUDO_FILES" | wc -w)"); do
        loc=$(echo "$SUDO_LOCATIONS" | cut -d' ' -f "$i")
        sudo rm "$loc" 2>/dev/null
        loc=$(echo "$loc" | rev | cut -d'/' -f2- | rev)
        while [ -z "ls -A $loc" ]; do
        while [ -z "$(ls -A "$loc")" ]; do
        sudo rm -r "$loc"
        loc=$(echo "$loc" | rev | cut -d'/' -f2- | rev)
        done 2>/dev/null
    done
    fi
    if [ "$(type -t custom)" = "function" ]; then
    custom remove
    if type custom | grep -q "function"; then
        custom remove
    fi
}

handle_package() {
    echo $1
    echo "$1"
    if [ -d "$1" ]; then
    unset DEPS
    eval $(grep "DEPS=" $1/DICT)
    if [ "$INSTALL" = "1" ] || ([ "$INSTALL" = "0" ] && [ "$REMOVE_DEPS" = "1" ]); then
    eval "$(grep "DEPS=" "$1/DICT")"
    if ! [ "$INSTALL" = "0" ] && [ "$REMOVE_DEPS" = "0" ]; then
        for i in $DEPS; do
            if ! grep -q "$i" "$PACKAGE_CACHE"; then
                (handle_package $i)
                (handle_package "$i")
            fi
        done
    fi
    echo "$1" >> "$PACKAGE_CACHE"
    (cd "$1"; "$([ \"$INSTALL\" = \"1\" ] && echo install || echo remove)_links")
    (cd "$1" || exit; "$([ "$INSTALL" = "1" ] && echo install || echo remove)_links")
    else
    echo "No configuration found for $i"
    fi
@@ -110,10 +110,10 @@
INSTALL=1
FORCE=0
REMOVE_DEPS=0
PACKAGE_CACHE=$(mktemp)
DOTFILES_PATH="$(dirname $0)"
PACKAGE_CACHE="$(mktemp)"
DOTFILES_PATH="$(dirname "$0")"

cd $DOTFILES_PATH
cd "$DOTFILES_PATH" || exit

if [ -z "$1" ]; then
    usage
@@ -138,4 +138,4 @@
    handle_package "$1"
done

rm $PACKAGE_CACHE
rm "$PACKAGE_CACHE"