Merge pull request #110 from kisslinux/audit

kiss: Add option to show diffs on update

FossilOrigin-Name: 61bad057ad3b31f1a7947f6801c9f618c6547c9e788b84b8f551f16a618eee65

1 files changed, 16 insertions, 2 deletions

diff --git a/kiss b/kiss
index 30d158d..f829f2f 100755
--- a/kiss
+++ b/kiss
@@ -1034,6 +1034,11 @@ pkg_updates() {
     # shellcheck disable=2046,2086
     { IFS=:; set -- $KISS_PATH; IFS=$old_ifs; }
 
+    # Where to store repository diffs for the update.
+    # At the same time, create the file so updates requiring root don't
+    # overwrite the user's existing permissions over the log.
+    :> "$mak_dir/log"
+
     # Update each repository in '$KISS_PATH'. It is assumed that
     # each repository is 'git' tracked.
     for repo; do
@@ -1065,6 +1070,7 @@ pkg_updates() {
 
             if [ -w "$PWD" ] && [ "$(id -u)" != 0 ]; then
                 git fetch
+                git diff >> "$mak_dir/log"
                 git merge
 
             else
@@ -1078,12 +1084,15 @@ pkg_updates() {
                 # case that the repository is owned by a 3rd user.
                 (
                     user=$(stat -c %U "$PWD")
+                    pull="git fetch && git diff >>'$mak_dir/log' && git merge"
 
                     [ "$user" = root ] ||
                         log "Dropping permissions to $user for pull"
 
-                    as_root git fetch
-                    as_root git merge
+                    case $su in
+                        su) as_root       "$pull" ;;
+                        *)  as_root sh -c "$pull" ;;
+                    esac
                 )
             fi
         }
@@ -1136,6 +1145,11 @@ pkg_updates() {
 
     log "Packages to update: ${outdated% }"
 
+    # Show a diff of each new change to the repositories.
+    # This spawns the user's set PAGER with a fallback to less.
+    [ -s "$mak_dir/log" ] && [ "$KISS_AUDIT" = 1 ] &&
+        "${PAGER:-less}" "$mak_dir/log"
+
     # Tell 'pkg_build' to always prompt before build.
     pkg_update=1