git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH] Install builtins with the user and group of the installing personality

From
Gerrit Pape <pape@smarden.org>
Date
Feb 24, 2009, 08:58 UTC
Message-ID
<20090224085817.16736.qmail@86bc3118539950.315fe32.mid.smarden.org>
In-Reply-To
<49A2D644.6040106@viscovery.net>

If 'make install' was run with sufficient privileges, then the installed builtins in gitexecdir, which are either hardlinked, symlinked, or copied, would receive the user and group of whoever built git. With this commit the initial hardlink or copy is done from the installation tree and not the build tree to fix this.

Signed-off-by: Gerrit Pape <pape@smarden.org>
---
Thanks, Johannes.
 Makefile |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile
index b040a96..0675c43 100644
--- a/Makefile
+++ b/Makefile
@@ -1468,8 +1468,8 @@ endif
 	bindir=$$(cd '$(DESTDIR_SQ)$(bindir_SQ)' && pwd) && \
 	execdir=$$(cd '$(DESTDIR_SQ)$(gitexec_instdir_SQ)' && pwd) && \
 	{ $(RM) "$$execdir/git-add$X" && \
-		ln git-add$X "$$execdir/git-add$X" 2>/dev/null || \
-		cp git-add$X "$$execdir/git-add$X"; } && \
+		ln "$$bindir/git$X" "$$execdir/git-add$X" 2>/dev/null || \
+		cp "$$bindir/git$X" "$$execdir/git-add$X"; } && \
 	{ for p in $(filter-out git-add$X,$(BUILT_INS)); do \
 		$(RM) "$$execdir/$$p" && \
 		ln "$$execdir/git-add$X" "$$execdir/$$p" 2>/dev/null || \
-- 
1.6.1.3
Previous: Johannes SixtNext: Junio C Hamano
Message 3 of 4 in “Install builtins with the user and group of the installing personality”
  1. Install builtins with the user and group of the installing personalityGerrit Pape, Feb 23, 2009
  2. Johannes SixtFeb 23, 2009
  3. Install builtins with the user and group of the installing personalityGerrit Pape, Feb 24, 2009
  4. Junio C HamanoFeb 25, 2009

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.