lmi-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi-commits] [lmi] odd/multiarch 60db190 01/16: Restore whitespace conc


From: Greg Chicares
Subject: [lmi-commits] [lmi] odd/multiarch 60db190 01/16: Restore whitespace concinnity
Date: Wed, 10 Apr 2019 22:15:06 -0400 (EDT)

branch: odd/multiarch
commit 60db1906e4890456dd37c988e4db14b21c4e1b0b
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Restore whitespace concinnity
    
    A recent change on this branch had altered the indentation of one
    'case' statement, making it more like other scripts but unlike the
    other 'case' statements in the same script. In retrospect, this did
    not seem to improve readability, and indenting this script's other
    'case' statements any further would cause them to exceed the
    aspirational eighty-character maximum width. Perhaps the 'case'
    keyword should induce only half a level of indenting as a general
    rule.
---
 check_git_setup.sh | 31 ++++++++++++++++---------------
 1 file changed, 16 insertions(+), 15 deletions(-)

diff --git a/check_git_setup.sh b/check_git_setup.sh
index 47c330c..27e85d0 100755
--- a/check_git_setup.sh
+++ b/check_git_setup.sh
@@ -34,23 +34,24 @@ printf '"%s" is git toplevel directory\n' "$toplevel"
 #'core.filemode' is "false". See:
 #   https://lists.nongnu.org/archive/html/lmi/2017-11/msg00018.html
 
+# Coordinate with changes in other repositories.
 # Call 'config.guess' here, or cache it in $LMI_BUILD?
 case "$LMI_BUILD" in
-    (*-*-cygwin*)
-        printf 'cygwin detected\n'
-        printf 'forcing correct permissions '
-          for d in . gwc; do (\
-               printf '%s...' "$d" \
-            && find ./$d -maxdepth 1 -type f \
-                -not -name '*.sh' -not -name '*.sed' \
-                | xargs chmod -x \
-          )done; \
-        printf 'all permissions forced\n'
-        git config core.filemode false
-        ;;
-    (*)
-        printf 'cygwin not detected--assuming OS is POSIX\n'
-        ;;
+  (*-*-cygwin*)
+    printf 'cygwin detected\n'
+    printf 'forcing correct permissions '
+      for d in . gwc; do (\
+           printf '%s...' "$d" \
+        && find ./$d -maxdepth 1 -type f \
+            -not -name '*.sh' -not -name '*.sed' \
+            | xargs chmod -x \
+      )done; \
+    printf 'all permissions forced\n'
+    git config core.filemode false
+    ;;
+  (*)
+    printf 'cygwin not detected--assuming OS is POSIX\n'
+    ;;
 esac
 
 printf 'core.filemode is "%s"\n' "$(git config --get-all core.filemode)"



reply via email to

[Prev in Thread] Current Thread [Next in Thread]