[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master 3dace32 05/14: Check permissions more thoroug
From: |
Greg Chicares |
Subject: |
[lmi-commits] [lmi] master 3dace32 05/14: Check permissions more thoroughly |
Date: |
Sat, 25 Apr 2020 17:43:12 -0400 (EDT) |
branch: master
commit 3dace3231709516bd5c6f90ed485710d96dfa6d5
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>
Check permissions more thoroughly
---
gwc/share_git1.sh | 14 +++++++++++++-
gwc/share_git2.sh | 3 +++
2 files changed, 16 insertions(+), 1 deletion(-)
diff --git a/gwc/share_git1.sh b/gwc/share_git1.sh
index 44551a2..3a58857 100755
--- a/gwc/share_git1.sh
+++ b/gwc/share_git1.sh
@@ -70,9 +70,21 @@ chmod g+ws .
# The crux of this method is 'git clone':
git clone --jobs=32 --bare --config core.SharedRepository=group
https://github.com/wxWidgets/zlib.git
+
+# Permissions seem to be okay...
+find ./zlib.git ! -perm -g=w |sed -e'/objects\/pack/d'
+# ...though FETCH_HEAD doesn't yet exist:
+ls -l ./zlib.git/*HEAD
+
# This succeeds when run by owner:
git -C zlib.git fetch
-# this succeeds (but not without 'umask 002' above):
+
+# Permissions seem to be okay...
+find ./zlib.git ! -perm -g=w |sed -e'/objects\/pack/d'
+# ...including FETCH_HEAD:
+ls -l ./zlib.git/*HEAD
+
+# This succeeds (but not without 'umask 002' above):
sudo --user=pulse git -C zlib.git fetch
# The two methods produce somewhat similar results. Sizes:
diff --git a/gwc/share_git2.sh b/gwc/share_git2.sh
index 377a820..383ee99 100755
--- a/gwc/share_git2.sh
+++ b/gwc/share_git2.sh
@@ -73,7 +73,10 @@ find zlib -type d -exec chmod g+s {} +
chmod -R g=u zlib
# Why doesn't 'g=u' override the earlier 'g+s'?
+# Permissions seem to be okay...
find ./zlib ! -perm -g=w |sed -e'/objects\/pack/d'
+# ...but that's because FETCH_HEAD doesn't yet exist:
+ls -l ./zlib/.git/*HEAD
# This succeeds when run by owner:
git -C zlib fetch
- [lmi-commits] [lmi] master 16a146d 03/14: Correct flaws in git script documentation, (continued)
- [lmi-commits] [lmi] master 16a146d 03/14: Correct flaws in git script documentation, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master 032d679 02/14: Reduce gratuitous differences between git scripts, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master 655214a 01/14: Note progress on git FETCH_HEAD permission issue, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master c78c9aa 06/14: Reduce differences between git scripts, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master 02f33b5 07/14: Don't rely on a nondefault umask, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master d432bc8 09/14: Improve documentation of git scripts, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master 28dbf67 08/14: Reduce differences between git scripts, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master 063d24d 04/14: Reduce differences between git scripts, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master 38a6c93 12/14: Let git create its own directory, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master 9f235b6 14/14: Don't mess with GID or permissions of $PWD, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master 3dace32 05/14: Check permissions more thoroughly,
Greg Chicares <=
- [lmi-commits] [lmi] master aedc8d0 11/14: Detect files that aren't group writable, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master 03f3b23 10/14: Reorder statements in a git script; improve documentation, Greg Chicares, 2020/04/25
- [lmi-commits] [lmi] master 7944d8e 13/14: Detect files that have an unexpected GID, Greg Chicares, 2020/04/25