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

Re: [PATCH] Fix build problems related to profile-directed optimization

From
Johannes Sixt <j.sixt@viscovery.net>
Date
Feb 9, 2012, 08:22 UTC
Message-ID
<4F338242.8080907@viscovery.net>
In-Reply-To
<20120208185319.GB9397@thunk.org>
Am 2/8/2012 19:53, schrieb Ted Ts'o:
> Junio, any comments on my most recent spin of this patch?  Any changes
> you'd like to see?
I need the following to unbreak my build on Windows.
--- >8 ---
From: Johannes Sixt <j6t@kdbg.org>
Subject: [PATCH] Makefile: fix syntax for older make

It is necessary to write the else branch as a nested conditional. Also, write the conditions with parentheses because we use them throughout the Makefile.

Signed-off-by: Johannes Sixt <j6t@kdbg.org>
---
 Makefile |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile
index bfc5daa..01a3c77 100644
--- a/Makefile
+++ b/Makefile
@@ -1784,16 +1784,18 @@ endif
 # data gathering
 PROFILE_DIR := $(CURDIR)
 
-ifeq "$(PROFILE)" "GEN"
+ifeq ("$(PROFILE)","GEN")
 	CFLAGS += -fprofile-generate=$(PROFILE_DIR) -DNO_NORETURN=1
 	EXTLIBS += -lgcov
 	export CCACHE_DISABLE=t
 	V=1
-else ifneq "$(PROFILE)" ""
+else
+ifneq ("$(PROFILE)","")
 	CFLAGS += -fprofile-use=$(PROFILE_DIR) -fprofile-correction -DNO_NORETURN=1
 	export CCACHE_DISABLE=t
 	V=1
 endif
+endif
 
 # Shell quote (do not use $(call) to accommodate ancient setups);
 
-- 
1.7.9.1420.gae2d6
Previous: Junio C HamanoNext: Junio C Hamano
Message 16 of 18 in “Fix build problems related to profile-directed optimization”
  1. Fix build problems related to profile-directed optimizationTheodore Ts'o, Feb 2, 2012
  2. Junio C HamanoFeb 2, 2012
  3. Ted Ts'oFeb 2, 2012
  4. Ted Ts'oFeb 2, 2012
  5. Junio C HamanoFeb 3, 2012
  6. Ted Ts'oFeb 3, 2012
  7. Junio C HamanoFeb 3, 2012
  8. Theodore TsoFeb 3, 2012
  9. Junio C HamanoFeb 3, 2012
  10. Fix build problems related to profile-directed optimizationTheodore Ts'o, Feb 6, 2012
  11. Jeff KingFeb 6, 2012
  12. Ted Ts'oFeb 6, 2012
  13. Fix build problems related to profile-directed optimizationTheodore Ts'o, Feb 6, 2012
  14. Ted Ts'oFeb 8, 2012
  15. Junio C HamanoFeb 9, 2012
  16. Johannes SixtFeb 9, 2012
  17. Junio C HamanoFeb 9, 2012
  18. Andi KleenFeb 3, 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.