emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH] Set also the group of installed files


From: Ulrich Mueller
Subject: [PATCH] Set also the group of installed files
Date: Thu, 21 Jun 2018 16:31:34 +0200

Currently the files in the lisp and etc directories are being
installed using a pair of tars, followed by a recursive chown.
This however adjusts only the user, but not the group ownership.

When building Emacs in a sandboxed environment (as it is the case
for Gentoo), the installed files will afterwards belong to root but
to the group of the build user. Note that this also doesn't match the
behaviour of install(1) which sets both user and group ownership.

See patch included below.

Ulrich


>From 851904d043ce268878444140df7eec8ff5de9a17 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ulrich=20M=C3=BCller?= <address@hidden>
Date: Thu, 21 Jun 2018 01:27:44 +0200
Subject: [PATCH] Set also the group of installed files

* Makefile.in (set_installuser): Also set the group, in order
to match install(1) behavior.
---
 Makefile.in | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/Makefile.in b/Makefile.in
index 52d44d9708..d659d7a56f 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -518,7 +518,10 @@ INSTALL_ARCH_INDEP_EXTRA = @INSTALL_ARCH_INDEP_EXTRA@
 set_installuser=for installuser in $${LOGNAME} $${USERNAME} $${USER} \
          `id -un 2> /dev/null`; do \
          [ -n "$${installuser}" ] && break ; \
-       done
+       done; \
+       installgroup=$$(id -gn 2>/dev/null); \
+       [ -n "$${installgroup}" ] && \
+         installuser="$${installuser}:$${installgroup}"
 
 ### Install the files that are machine-independent.
 ### Most of them come straight from the distribution; the exception is
-- 
2.17.1



reply via email to

[Prev in Thread] Current Thread [Next in Thread]