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

[PATCH] Makefile: limit specifying -DGIT_USER_AGENT=... to keep ccache efficient

From
Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Date
Jun 24, 2012, 12:37 UTC
Message-ID
<1340541457-9056-1-git-send-email-pclouds@gmail.com>

GIT_USER_AGENT changes as HEAD changes. Defining it in BASIC_FLAGS means every time HEAD changes, the compiling options for every object file is changed, which defeats the purpose of using ccache.

As version.c is the only file that uses this definition, limit defining only when compiling this file.

Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
---
 Makefile |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile
index f62ca2a..b3ce00d 100644
--- a/Makefile
+++ b/Makefile
@@ -1929,7 +1929,6 @@ endif
 GIT_USER_AGENT_SQ = $(subst ','\'',$(GIT_USER_AGENT))
 GIT_USER_AGENT_CQ = "$(subst ",\",$(subst \,\\,$(GIT_USER_AGENT)))"
 GIT_USER_AGENT_CQ_SQ = $(subst ','\'',$(GIT_USER_AGENT_CQ))
-BASIC_CFLAGS += -DGIT_USER_AGENT='$(GIT_USER_AGENT_CQ_SQ)'
 
 ifdef DEFAULT_HELP_FORMAT
 BASIC_CFLAGS += -DDEFAULT_HELP_FORMAT='"$(DEFAULT_HELP_FORMAT)"'
@@ -2000,7 +1999,8 @@ builtin/help.sp builtin/help.s builtin/help.o: EXTRA_CPPFLAGS = \
 	'-DGIT_INFO_PATH="$(infodir_SQ)"'
 
 version.sp version.s version.o: EXTRA_CPPFLAGS = \
-	'-DGIT_VERSION="$(GIT_VERSION)"'
+	'-DGIT_VERSION="$(GIT_VERSION)"' \
+	'-DGIT_USER_AGENT="$(GIT_USER_AGENT)"'
 
 $(BUILT_INS): git$X
 	$(QUIET_BUILT_IN)$(RM) $@ && \
-- 
1.7.8
Next: Nguyen Thai Ngoc Duy
Message 1 of 2 in “Makefile: limit specifying -DGIT_USER_AGENT=... to keep ccache efficient”
  1. Makefile: limit specifying -DGIT_USER_AGENT=... to keep ccache efficientNguyễn Thái Ngọc Duy, Jun 24, 2012
  2. Nguyen Thai Ngoc DuyJun 24, 2012

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.