Fix typo

Co-authored-by: 0x1eef <0x1eef@protonmail.com>
Reviewed-on: http://git.bastion.home.network/0x1eef/sourcezap/pulls/7

libexec/sourcezap/commands/sourcezap-checkout

@@ -35,4 +35,4 @@ cd "${gitdir}"
 "${git}" fetch origin
 "${git}" checkout "${branch}" ||
 "${git}" checkout -t origin/"${branch}"
-printok "${branch} checked out"
+printok "checkout complete"

libexec/sourcezap/commands/sourcezap-install

@@ -5,7 +5,6 @@ set -e
 # variables
 localbase=${LOCALBASE:-$(realpath "$(dirname "$0")"/../../..)}
 libexec="${localbase}"/libexec/sourcezap
-mode=u=rwX,g=rX,o=
 git="${libexec}"/utils/git/run
 gitdir=$1
 installdir=$2
@@ -16,35 +15,26 @@ revfile=$3
 # shellcheck source=/dev/null
 . "${libexec}"/functions/print.sh
 
-perform_update()
+##
-{
+# main
-    rev=$(cat "${revfile}")
+if [ "$(id -u)" != "0" ]; then
-    add=$("${libexec}"/utils/git/get-changed-files "${gitdir}" "${rev}")
+    printerr "you must be root"
-    del=$("${libexec}"/utils/git/get-removed-files "${gitdir}" "${rev}")
+    exit 1
-    for file in ${del}; do
+fi
-        target="${installdir}/${file}"
-        parent=$(dirname "${target}")
-        echo "rm ${target}"
-        rm -f "${target}"
-        find "${parent}" -type d -maxdepth 0 -empty -delete
-    done
-    for file in ${add}; do
-        target="${installdir}/${file}"
-        parent=$(dirname "${target}")
-        parents=""
-        while [ ! -e "${parent}" ]; do
-            parents="${parent} ${parents}"
-            parent=$(dirname "${parent}")
-        done
-        for dir in ${parents}; do
-            run_install "-d" "${dir}"
-        done
-        run_install "${file}" "${target}"
-    done
-}
 
-perform_install()
+cd "${gitdir}"
-{
+if [ -e "${revfile}" ]; then
+    ##
+    # install update
+    rev=$(cat "${revfile}")
+    "${libexec}"/utils/install/run -d "${installdir}"
+    "${libexec}"/utils/install/update-deleted-files "${gitdir}" "${installdir}" "${rev}"
+    "${libexec}"/utils/install/update-changed-files "${gitdir}" "${installdir}" "${rev}"
+else
+    ##
+    # install from scratch
+    umask u=rwX,g=rX,o=
+    "${libexec}"/utils/install/run -d "${installdir}"
     find -s . \
          -maxdepth 1 \
          ! -name "." \
@@ -60,28 +50,6 @@ perform_install()
          ! -name ".arcconfig" \
          -exec cp -Rpv {} "${installdir}" \;
     chown -Rv root "${installdir}"
-}
-
-run_install()
-{
-    install -o root -g _sourcezap -m "${mode}" -v "$@"
-}
-
-##
-# main
-if [ "$(id -u)" != "0" ]; then
-    printerr "you must be root"
-    exit 1
-fi
-
-umask "${mode}"
-cd "${gitdir}"
-run_install "-d" "${installdir}"
-chmod ${mode} "${installdir}"
-if [ -e "${revfile}" ]; then
-    perform_update
-else
-    perform_install
 fi
 "${git}" rev-parse HEAD > "${revfile}"
 printok "install complete"

libexec/sourcezap/commands/sourcezap-rm

@@ -15,6 +15,8 @@ installdir=$2
 
 erase()
 {
+    # shellcheck disable=SC3043
+    local dir
     dir="${1}"
     printf "%s  " "${dir}"
     find "${dir}" \

libexec/sourcezap/commands/sourcezap-setup

@@ -16,6 +16,7 @@ libexec="${localbase}"/libexec/sourcezap
 if [ "$(id -u)" = "0" ]; then
     "${libexec}"/setup/setup-user
     "${libexec}"/setup/setup-doas
+    printok "setup complete"
 else
     printerr "you must be root"
     exit 1

libexec/sourcezap/commands/sourcezap-teardown

@@ -15,10 +15,10 @@ user=_sourcezap
 ##
 # main
 if [ "$(id -u)" = "0" ]; then
-    pw userdel  -n "${user}"  || true
+    pw userdel  -n "${user}"  > /dev/null 2>&1 || true
-    pw groupdel -n "${user}"  || true
+    pw groupdel -n "${user}"  > /dev/null 2>&1 || true
-    rm -rf /home/"${user:?}"/ || true
+    rm -rf /home/"${user:?}"/ > /dev/null 2>&1 || true
-    printok "done"
+    printok "teardown complete"
 else
     printerr "you must be root"
     exit 1

libexec/sourcezap/utils/install/run

@@ -0,0 +1,14 @@
+#!/bin/sh
+set -e
+
+##
+# variables
+mode=u=rwX,g=rX,o=
+
+##
+# main
+install -o root \
+        -g _sourcezap \
+        -m "${mode}" \
+        -v \
+        "${@}"

libexec/sourcezap/utils/install/update-changed-files

@@ -0,0 +1,28 @@
+#!/bin/sh
+set -e
+
+##
+# variables
+localbase=${LOCALBASE:-$(realpath "$(dirname "$0")"/../../../..)}
+libexec="${localbase}"/libexec/sourcezap
+gitdir=$1
+installdir=$2
+rev=$3
+
+##
+# main
+cd "${gitdir}"
+files=$("${libexec}"/utils/git/get-changed-files "${gitdir}" "${rev}")
+for file in ${files}; do
+    target="${installdir}/${file}"
+    parent=$(dirname "${target}")
+    parents=""
+    while [ ! -e "${parent}" ]; do
+        parents="${parent} ${parents}"
+        parent=$(dirname "${parent}")
+    done
+    for dir in ${parents}; do
+        "${libexec}"/utils/install/run -d "${dir}"
+    done
+    "${libexec}"/utils/install/run "${file}" "${target}"
+done

libexec/sourcezap/utils/install/update-deleted-files

@@ -0,0 +1,22 @@
+#!/bin/sh
+set -e
+
+##
+# variables
+localbase=${LOCALBASE:-$(realpath "$(dirname "$0")"/../../../..)}
+libexec="${localbase}"/libexec/sourcezap
+gitdir=$1
+installdir=$2
+rev=$3
+
+##
+# main
+cd "${gitdir}"
+files=$("${libexec}"/utils/git/get-removed-files "${gitdir}" "${rev}")
+for file in ${files}; do
+    target="${installdir}/${file}"
+    parent=$(dirname "${target}")
+    echo "rm ${target}"
+    rm -f "${target}"
+    find "${parent}" -type d -maxdepth 0 -empty -delete
+done

share/sourcezap/CHANGELOG

@@ -1,13 +1,17 @@
 * vNEXT
 
+** Break up 'sourcezap-install' into multiple files
+'libexec/sourcezap/utils/install/' contains files that cover both
+a fresh install and applying an update
+
 ** Add 'setup/setup-doas' improvements
 More likely to do what's expected, but blind spots still exist
 
 ** Add 'sourcezap setup', 'sourcezap teardown'
 Replaces and enhances 'setup-sourcezap'
 
-** Add libexec/sourcezap/commands/sourcezap-sh
+** Add 'libexec/sourcezap/commands/sourcezap-sh'
-Runs /bin/sh within /home/_sourcezap/src/ as the '_sourcezap' user
+Runs '/bin/sh' within '/home/_sourcezap/src/' as the '_sourcezap' user
 
 * v1.0.0