emacs-diffs
[Top][All Lists]
Advanced

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

[Emacs-diffs] master 9a27310: Set group when installing, too


From: Paul Eggert
Subject: [Emacs-diffs] master 9a27310: Set group when installing, too
Date: Thu, 21 Jun 2018 16:30:30 -0400 (EDT)

branch: master
commit 9a27310f0ab068bef52cfe30d3abc5eda3a0d12b
Author: Paul Eggert <address@hidden>
Commit: Paul Eggert <address@hidden>

    Set group when installing, too
    
    From a patch by Ulrich Mueller in:
    https://lists.gnu.org/r/emacs-devel/2018-06/msg00687.html
    * Makefile.in (set_installuser): Also set the group, in order
    to match install(1) behavior.  Also, don’t clutter stderr
    with a diagnostic if ‘id’ is missing.
---
 Makefile.in | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/Makefile.in b/Makefile.in
index 52d44d9..4d7627b 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -516,9 +516,11 @@ INSTALL_ARCH_INDEP_EXTRA = @INSTALL_ARCH_INDEP_EXTRA@
 ## https://lists.gnu.org/r/emacs-devel/2007-10/msg01672.html
 ## Needs to be the user running install, so configure can't set it.
 set_installuser=for installuser in $${LOGNAME} $${USERNAME} $${USER} \
-         `id -un 2> /dev/null`; do \
+         `(id -u) 2> /dev/null`; do \
          [ -n "$${installuser}" ] && break ; \
-       done
+       done; \
+       installgroup=`(id -g) 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



reply via email to

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