[PATCH] libtraceevent: remove hard coded include to /usr/local/include in Makefile

[PATCH] libtraceevent: remove hard coded include to /usr/local/include in Makefile

[Jack Mitchell]

From: Jack Mitchell <jack.mitchell@dbbroadcast.co.uk>

having /usr/local/include hardcoded into the makefile is not necessary as this
is automatically included by GCC. It also infects cross-compile builds with the
host systems includes.

Signed-off-by: Jack Mitchell <jack.mitchell@dbbroadcast.co.uk>
---
 tools/lib/traceevent/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/lib/traceevent/Makefile b/tools/lib/traceevent/Makefile
index a20e320..0b0a907 100644
--- a/tools/lib/traceevent/Makefile
+++ b/tools/lib/traceevent/Makefile
@@ -122,7 +122,7 @@ export Q VERBOSE
 
 EVENT_PARSE_VERSION = $(EP_VERSION).$(EP_PATCHLEVEL).$(EP_EXTRAVERSION)
 
-INCLUDES = -I. -I/usr/local/include $(CONFIG_INCLUDES)
+INCLUDES = -I. $(CONFIG_INCLUDES)
 
 # Set compile option CFLAGS if not set elsewhere
 CFLAGS ?= -g -Wall
-- 
1.8.1.5

Re: [PATCH] libtraceevent: remove hard coded include to /usr/local/include in Makefile

[Namhyung Kim]

On Fri,  8 Mar 2013 11:21:52 +0000, Jack Mitchell wrote:
> From: Jack Mitchell <jack.mitchell@dbbroadcast.co.uk>
>
> having /usr/local/include hardcoded into the makefile is not necessary as this
> is automatically included by GCC. It also infects cross-compile builds with the
> host systems includes.

Acked-by: Namhyung Kim <namhyung@kernel.org>

Thanks,
Namhyung

[tip:perf/urgent] libtraceevent: Remove hard coded include to / usr/local/include in Makefile

[tip-bot for Jack Mitchell]

Commit-ID:  b9e8c37220c80e78289a1e87b50c09418eb59a7e
Gitweb:     http://git.kernel.org/tip/b9e8c37220c80e78289a1e87b50c09418eb59a7e
Author:     Jack Mitchell <jack.mitchell@dbbroadcast.co.uk>
AuthorDate: Fri, 8 Mar 2013 11:21:52 +0000
Committer:  Arnaldo Carvalho de Melo <acme@redhat.com>
CommitDate: Wed, 13 Mar 2013 17:00:00 -0300

libtraceevent: Remove hard coded include to /usr/local/include in Makefile

having /usr/local/include hardcoded into the makefile is not necessary
as this is automatically included by GCC. It also infects cross-compile
builds with the host systems includes.

Signed-off-by: Jack Mitchell <jack.mitchell@dbbroadcast.co.uk>
Acked-by: Namhyung Kim <namhyung@kernel.org>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1362741712-21308-1-git-send-email-ml@communistcode.co.uk
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/lib/traceevent/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/lib/traceevent/Makefile b/tools/lib/traceevent/Makefile
index a20e320..0b0a907 100644
--- a/tools/lib/traceevent/Makefile
+++ b/tools/lib/traceevent/Makefile
@@ -122,7 +122,7 @@ export Q VERBOSE
 
 EVENT_PARSE_VERSION = $(EP_VERSION).$(EP_PATCHLEVEL).$(EP_EXTRAVERSION)
 
-INCLUDES = -I. -I/usr/local/include $(CONFIG_INCLUDES)
+INCLUDES = -I. $(CONFIG_INCLUDES)
 
 # Set compile option CFLAGS if not set elsewhere
 CFLAGS ?= -g -Wall