This is an automated email from the ASF dual-hosted git repository.

root pushed a commit to branch tlater/casd-socket-permissions
in repository https://gitbox.apache.org/repos/asf/buildstream.git

commit 397ddcfe7a728aec06307ee24f7c8be05130f490
Author: Tristan Maat <[email protected]>
AuthorDate: Fri Nov 8 12:24:43 2019 +0000

    plugins/sources/tar.py: Ensure read permissions for groups as well
---
 src/buildstream/plugins/sources/tar.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/buildstream/plugins/sources/tar.py 
b/src/buildstream/plugins/sources/tar.py
index 702b7ba..ac80177 100644
--- a/src/buildstream/plugins/sources/tar.py
+++ b/src/buildstream/plugins/sources/tar.py
@@ -69,14 +69,14 @@ from ._downloadablefilesource import DownloadableFileSource
 class ReadableTarInfo(tarfile.TarInfo):
     """
            The goal is to override `TarFile`'s `extractall` semantics by 
ensuring that on extraction, the
-           files are readable by the owner of the file. This is done by 
overriding the accessor for the
+           files are readable by the owners of the file. This is done by 
overriding the accessor for the
            `mode` attribute in `TarInfo`, the class that encapsulates the 
internal meta-data of the tarball,
            so that the owner-read bit is always set.
     """
     @property
     def mode(self):
-        # ensure file is readable by owner
-        return self.__permission | 0o400
+        # ensure file is readable by owner and group
+        return self.__permission | 0o440
 
     @mode.setter
     def mode(self, permission):

Reply via email to