[lmi-commits] [lmi] master c408c8c6: Move some new commands that now require su

[Greg Chicares]

branch: master
commit c408c8c66de432e173b55f1d4d01e37ee13da515
Author: Gregory W. Chicares <gchicares@sbcglobal.net>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>

    Move some new commands that now require su
    
    Forcing ownership and permissions of all files in the proprietary
    repository's directory /srv/cache_for_lmi/blessed/ may require
    superuser powers now that that directory resides on the host.
---
 lmi_setup_29.sh | 19 ++++++++++++++++++-
 lmi_setup_43.sh | 19 +------------------
 2 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/lmi_setup_29.sh b/lmi_setup_29.sh
index 52291683..a1b86d83 100755
--- a/lmi_setup_29.sh
+++ b/lmi_setup_29.sh
@@ -46,5 +46,22 @@ mkdir --parents /srv/cache_for_lmi
 chmod g=u+s     /srv/cache_for_lmi
 chown "${NORMAL_USER}":"${NORMAL_GROUP}" /srv/cache_for_lmi
 
+# To duplicate proprietary repository (if available) from another
+# machine to the host's /srv/cache_for_lmi/ , which has been
+# identity-mounted in the chroot (here, 'cp' is sufficient--this
+# bare repository has no references that need to be resolved):
+# cp --dereference --preserve --recursive WHENCEVER/blessed 
/srv/cache_for_lmi/blessed
+
+# Fix ownership and permissions of bare repository, just in case.
+chgrp -R "$NORMAL_GROUP" /srv/cache_for_lmi/blessed
+# This is better than 'chmod -R g+s' (it affects only directories):
+find /srv/cache_for_lmi/blessed -type d -exec chmod g+s {} +
+# Specifying 's' here would cause many 'S' occurrences in 'ls' output;
+# specifying 'g+w' here would cause pack files to be group writable:
+#   chmod -R g+swX /srv/cache_for_lmi/blessed
+# Instead, use 'g=u', which doesn't override the earlier 'g+s'--see:
+#   https://lists.nongnu.org/archive/html/lmi/2020-03/msg00019.html
+chmod -R g=u /srv/cache_for_lmi/blessed
+
 stamp=$(date -u +'%Y%m%dT%H%M%SZ')
-echo "$stamp $0: Created lmi directories."  | tee /dev/tty
+echo "$stamp $0: Created lmi directories; adjusted git."  | tee /dev/tty
diff --git a/lmi_setup_43.sh b/lmi_setup_43.sh
index 8841454c..af71afc3 100755
--- a/lmi_setup_43.sh
+++ b/lmi_setup_43.sh
@@ -41,24 +41,7 @@ if [ "greg" = "$(whoami)" ]; then
   git remote set-url --push origin chicares@git.sv.gnu.org:/srv/git/lmi.git
 fi
 
-# Duplicate proprietary repository (if available) from another
-# machine to the host's /srv/cache_for_lmi/ , which has been
-# identity-mounted in the chroot. Here, 'cp' is sufficient (this
-# bare repository has no references that need to be resolved):
-# cp --dereference --preserve --recursive WHENCEVER/blessed 
/srv/cache_for_lmi/blessed
-#
-# Fix ownership and permissions of bare repository, just in case.
-chgrp -R "$NORMAL_GROUP" /srv/cache_for_lmi/blessed
-# This is better than 'chmod -R g+s' (it affects only directories):
-find /srv/cache_for_lmi/blessed -type d -exec chmod g+s {} +
-# Specifying 's' here would cause many 'S' occurrences in 'ls' output;
-# specifying 'g+w' here would cause pack files to be group writable:
-#   chmod -R g+swX /srv/cache_for_lmi/blessed
-# Instead, use 'g=u', which doesn't override the earlier 'g+s'--see:
-#   https://lists.nongnu.org/archive/html/lmi/2020-03/msg00019.html
-chmod -R g=u /srv/cache_for_lmi/blessed
-#
-# Then create a working copy by cloning the bare repository...
+# Clone the bare proprietary repository to create a working copy.
 #
 # Apparently '--config core.SharedRepository=group' would have little
 # or no benefit here.