From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760213Ab2IFCL4 (ORCPT ); Wed, 5 Sep 2012 22:11:56 -0400 Received: from mail.windriver.com ([147.11.1.11]:38762 "EHLO mail.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752979Ab2IFCLz (ORCPT ); Wed, 5 Sep 2012 22:11:55 -0400 Date: Thu, 6 Sep 2012 10:11:22 +0800 From: Liang Li To: Arnaldo Carvalho de Melo CC: Richard Purdie , , , , Subject: Re: [RFC PATCH] perf: add SLANG_INC for slang.h Message-ID: <20120906021121.GB16647@localhost> Reply-To: Liang Li References: <1345777839-15843-1-git-send-email-liang.li@windriver.com> <20120905205439.GD8203@ghostprotocols.net> <1346880183.21985.90.camel@ted> <20120905224122.GH8203@ghostprotocols.net> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20120905224122.GH8203@ghostprotocols.net> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2012-09-06 06:41, Arnaldo Carvalho de Melo wrote: > Em Wed, Sep 05, 2012 at 10:23:03PM +0100, Richard Purdie escreveu: > > On Wed, 2012-09-05 at 13:54 -0700, Arnaldo Carvalho de Melo wrote: > > > Em Fri, Aug 24, 2012 at 11:10:39AM +0800, Liang Li escreveu: > > > > CFLAGS was previously hard coded to contain "-I/usr/include/slang" to > > > > work with hosts that have "/usr/include/slang/slang.h" as well as hosts > > > > that have "/usr/include/slang.h". This path can cause compile warnings > > > > like: > > > > > > > > cc1: warning: '/usr/include/slang' doesn't exists. > > > > > > > > or > > > > > > > > cc1: warning: include location "/usr/include/slang" is unsafe for > > > > cross-compilation [-Wpoison-system-directories] > > > > > > > > Then in some cases warnings become errors if WERROR is enabled hence > > > > build errors. > > > > > > > > To fix this issue, we can use -idirafter to downgrade the priority of the > > > > default hard coded path. We can also make the slang include directory > > > > a variable, to allow the user to specify SLANG_INC and set their own > > > > include location. And add a '=' prefix to indicate better > > > > compatibility with sysroot/cross compile cases. > > > > > > CC /home/acme/git/build/perf/builtin-diff.o > > > In file included from util/../ui/keysyms.h:4, > > > from util/hist.h:142, > > > from builtin-diff.c:11: > > > util/../ui/libslang.h:12:19: error: slang.h: No such file or directory > > > util/../ui/libslang.h:14:5: error: "SLANG_VERSION" is not defined > > > In file included from util/../ui/keysyms.h:4, > > > from util/hist.h:142, > > > from util/evsel.h:10, > > > from util/evlist.h:8, > > > from builtin-annotate.c:20: > > > util/../ui/libslang.h:12:19: error: slang.h: No such file or directory > > > util/../ui/libslang.h:14:5: error: "SLANG_VERSION" is not defined > > > builtin-annotate.c: In function ‘hists__find_annotations’: > > > builtin-annotate.c:122: error: ‘SL_KEY_RIGHT’ undeclared (first use in > > > this function) > > > builtin-annotate.c:122: error: (Each undeclared identifier is reported > > > only once > > > builtin-annotate.c:122: error: for each function it appears in.) > > > builtin-annotate.c:134: error: ‘SL_KEY_LEFT’ undeclared (first use in > > > this function) > > > make: *** [/home/acme/git/build/perf/builtin-annotate.o] Error 1 > > > make: *** Waiting for unfinished jobs.... > > > make: *** [/home/acme/git/build/perf/builtin-diff.o] Error 1 > > > make: Leaving directory `/home/git/linux/tools/perf' > > > [acme@sandy linux]$ cat /etc/redhat-release > > > Red Hat Enterprise Linux Server release 6.3 (Santiago) > > > [acme@sandy linux]$ > > > > Does: > > > > SLANG_INC ?= -I=/usr/include/slang > > > > work any better? Its hard to tell from the above error which bit of the > > I'll try later > > > syntax is failing. Which gcc version is it? > > [acme@sandy linux]$ gcc -v > Using built-in specs. > Target: x86_64-redhat-linux > Configured with: ../configure --prefix=/usr --mandir=/usr/share/man > --infodir=/usr/share/info > --with-bugurl=http://bugzilla.redhat.com/bugzilla --enable-bootstrap > --enable-shared --enable-threads=posix --enable-checking=release > --with-system-zlib --enable-__cxa_atexit --disable-libunwind-exceptions > --enable-gnu-unique-object > --enable-languages=c,c++,objc,obj-c++,java,fortran,ada > --enable-java-awt=gtk --disable-dssi > --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-1.5.0.0/jre > --enable-libgcj-multifile --enable-java-maintainer-mode > --with-ecj-jar=/usr/share/java/eclipse-ecj.jar > --disable-libjava-multilib --with-ppl --with-cloog --with-tune=generic > --with-arch_32=i686 --build=x86_64-redhat-linux > Thread model: posix > gcc version 4.4.6 20120305 (Red Hat 4.4.6-4) (GCC) > [acme@sandy linux]$ Seems like there is no slang.h installed. Could you please check: $ find /usr/include/ -name slang.h On my FC17: $ find /usr/include/ -name slang.h /usr/include/slang/slang.h /usr/include/slang.h $ rpm -qf /usr/include/slang.h slang-devel-2.2.4-3.fc17.x86_64 $ rpm -qf /usr/include/slang/slang.h slang-devel-2.2.4-3.fc17.x86_64 --- And does 'export SLANG_INC=' works any better? :) Cheers, Liang Li