update-all-config/update-all-hooks: disable hook a+rx checkingmaster
authorKyle J. McKay <mackyle@gmail.com>
Wed, 29 Apr 2015 13:52:14 +0000 (29 06:52 -0700)
committerKyle J. McKay <mackyle@gmail.com>
Wed, 29 Apr 2015 13:52:14 +0000 (29 06:52 -0700)
Old versions of Git may have installed sample hooks without the
".sample" extension that are not-executable instead to mark them
as being "samples".  We would try to make those a+rx which is
undesirable.

Disable the a+rx hooks checking code for now.

Signed-off-by: Kyle J. McKay <mackyle@gmail.com>
toolbox/update-all-config.sh
toolbox/update-all-hooks.sh

index 51dcb37..19b6fc8 100755 (executable)
@@ -347,7 +347,8 @@ eval "$cmd" | \
                        $(find . -xdev -type d \( -path ./hooks -o -path ./mob/hooks \) -prune -o -type f -perm +a+x -print)
                EOT
                while read -r fixfilex; do
-                       case "$fixfilex" in ""|*.sample) :;; *) change_fpermx "$fixfilex"; esac
+                       #case "$fixfilex" in ""|*.sample) :;; *) change_fpermx "$fixfilex"; esac
+                       :
                done <<-EOT
                        $(! [ -d hooks ] || find hooks -xdev -type f -not -perm -a+x -print
                        ! [ -d mob/hooks ] || find mob/hooks -xdev -type f -not -perm -a+x -print)
index 951cb36..452f6f7 100755 (executable)
@@ -87,8 +87,9 @@ eval "$cmd" | \
                added=
                while read -r fixfilex; do
                        case "$fixfilex" in ""|*.sample) :;; *)
-                               [ -n "$added" ] || { added=1; updates="$updates a+rx"; }
-                               [ -n "$dryrun" ] || chmod a+rx "$projdir/$fixfilex"
+                               #[ -n "$added" ] || { added=1; updates="$updates a+rx"; }
+                               #[ -n "$dryrun" ] || chmod a+rx "$projdir/$fixfilex"
+                               :
                        esac
                done <<-EOT
                        $(cd "$projdir"