Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753253Ab2HTCUH (ORCPT ); Sun, 19 Aug 2012 22:20:07 -0400 Received: from LGEMRELSE7Q.lge.com ([156.147.1.151]:65134 "EHLO LGEMRELSE7Q.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753035Ab2HTCUE (ORCPT ); Sun, 19 Aug 2012 22:20:04 -0400 X-AuditID: 9c930197-b7bb2ae0000011d9-0c-50319ed2047a From: Namhyung Kim To: David Ahern Cc: acme@ghostprotocols.net, linux-kernel@vger.kernel.org, Pekka Enberg Subject: Re: [PATCH] perf: silence GTK2 probing errors References: <1345391202-71865-1-git-send-email-dsahern@gmail.com> Date: Mon, 20 Aug 2012 11:13:07 +0900 In-Reply-To: <1345391202-71865-1-git-send-email-dsahern@gmail.com> (David Ahern's message of "Sun, 19 Aug 2012 09:46:42 -0600") Message-ID: <87vcgeia4s.fsf@sejong.aot.lge.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Brightmail-Tracker: AAAAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3425 Lines: 79 Hi, David On Sun, 19 Aug 2012 09:46:42 -0600, David Ahern wrote: > If GTK2 development packages are not installed, make is rather noisy: > > $ make -C tools/perf O=/tmp/pbuild > Package gtk+-2.0 was not found in the pkg-config search path. > Perhaps you should add the directory containing `gtk+-2.0.pc' > to the PKG_CONFIG_PATH environment variable > No package 'gtk+-2.0' found > make: Entering directory `/opt/sw/ahern/perf.git/tools/perf' > Makefile:593: GTK2 not found, disables GTK2 support. Please install gtk2-devel or libgtk2.0-dev > PERF_VERSION = 3.6.rc1.205.gdb146f.dirty > make: Leaving directory `/opt/sw/ahern/perf.git/tools/perf' > Package gtk+-2.0 was not found in the pkg-config search path. > Perhaps you should add the directory containing `gtk+-2.0.pc' > to the PKG_CONFIG_PATH environment variable > No package 'gtk+-2.0' found > make: Entering directory `/opt/sw/ahern/perf.git/tools/perf' > Makefile:593: GTK2 not found, disables GTK2 support. Please install gtk2-devel or libgtk2.0-dev > ... > > Silence the pkg-config errors. Aftewards: > > $ make -C tools/perf O=/tmp/pbuild > make: Entering directory `/opt/sw/ahern/perf.git/tools/perf' > Makefile:593: GTK2 not found, disables GTK2 support. Please install gtk2-devel or libgtk2.0-dev > PERF_VERSION = 3.6.rc1.206.gd43ff9.dirty > make: Leaving directory `/opt/sw/ahern/perf.git/tools/perf' > make: Entering directory `/opt/sw/ahern/perf.git/tools/perf' > Makefile:593: GTK2 not found, disables GTK2 support. Please install gtk2-devel or libgtk2.0-dev > ... The patch looks good to me. But I wonder why we see this "GTK2 not found" message twice even in -j1 build. Perhaps we need to define the NO_GTK2 after the feature-test. Thanks, Namhyung > > Signed-off-by: David Ahern > Cc: Arnaldo Carvalho de Melo > Cc: Pekka Enberg > Cc: Namhyung Kim > --- > tools/perf/Makefile | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/tools/perf/Makefile b/tools/perf/Makefile > index 1eb7f12..90cfecf 100644 > --- a/tools/perf/Makefile > +++ b/tools/perf/Makefile > @@ -588,7 +588,7 @@ endif > ifdef NO_GTK2 > BASIC_CFLAGS += -DNO_GTK2_SUPPORT > else > - FLAGS_GTK2=$(ALL_CFLAGS) $(ALL_LDFLAGS) $(EXTLIBS) $(shell pkg-config --libs --cflags gtk+-2.0) > + FLAGS_GTK2=$(ALL_CFLAGS) $(ALL_LDFLAGS) $(EXTLIBS) $(shell pkg-config --libs --cflags gtk+-2.0 2>/dev/null) > ifneq ($(call try-cc,$(SOURCE_GTK2),$(FLAGS_GTK2)),y) > msg := $(warning GTK2 not found, disables GTK2 support. Please install gtk2-devel or libgtk2.0-dev); > BASIC_CFLAGS += -DNO_GTK2_SUPPORT > @@ -596,8 +596,8 @@ else > ifeq ($(call try-cc,$(SOURCE_GTK2_INFOBAR),$(FLAGS_GTK2)),y) > BASIC_CFLAGS += -DHAVE_GTK_INFO_BAR > endif > - BASIC_CFLAGS += $(shell pkg-config --cflags gtk+-2.0) > - EXTLIBS += $(shell pkg-config --libs gtk+-2.0) > + BASIC_CFLAGS += $(shell pkg-config --cflags gtk+-2.0 2>/dev/null) > + EXTLIBS += $(shell pkg-config --libs gtk+-2.0 2>/dev/null) > LIB_OBJS += $(OUTPUT)ui/gtk/browser.o > LIB_OBJS += $(OUTPUT)ui/gtk/setup.o > LIB_OBJS += $(OUTPUT)ui/gtk/util.o -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/