about summary refs log tree commit homepage
path: root/lib
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2018-05-30 02:54:48 +0000
committerEric Wong <e@80x24.org>2018-05-30 20:34:23 +0000
commit6ca8f8d03fa8ee93edc8299f53d624f5d5841408 (patch)
tree762802e0818b5a4beac41bd2b12b432e73987907 /lib
parentc339a14b11c57e7b330f4efef6903bcbea2b3463 (diff)
downloadpublic-inbox-6ca8f8d03fa8ee93edc8299f53d624f5d5841408.tar.gz
This is consistent with git itself and the previous behavior
was a result of misunderstanding of how git interprets this.
And adjust tests slightly to match the new behavior.

Reported-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
	<38873789-ab42-65a1-20c9-12c30b171f4f@linuxfoundation.org>
Diffstat (limited to 'lib')
-rw-r--r--lib/PublicInbox/InboxWritable.pm2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/InboxWritable.pm b/lib/PublicInbox/InboxWritable.pm
index 5c11a36c..9b0cdfd0 100644
--- a/lib/PublicInbox/InboxWritable.pm
+++ b/lib/PublicInbox/InboxWritable.pm
@@ -175,7 +175,7 @@ sub _read_git_config_perm {
 sub _git_config_perm {
         my $self = shift;
         my $perm = scalar @_ ? $_[0] : _read_git_config_perm($self);
-        return PERM_GROUP if (!defined($perm) || $perm eq '');
+        return PERM_UMASK if (!defined($perm) || $perm eq '');
         return PERM_UMASK if ($perm eq 'umask');
         return PERM_GROUP if ($perm eq 'group');
         if ($perm =~ /\A(?:all|world|everybody)\z/) {