Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1751921AbbD2TOg (ORCPT <rfc822;w@1wt.eu>);
	Wed, 29 Apr 2015 15:14:36 -0400
Received: from mx1.redhat.com ([209.132.183.28]:40685 "EHLO mx1.redhat.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1750946AbbD2TOd (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
	Wed, 29 Apr 2015 15:14:33 -0400
Date: Wed, 29 Apr 2015 21:14:26 +0200
From: Jiri Olsa <jolsa@redhat.com>
To: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>, Ingo Molnar <mingo@kernel.org>,
        Peter Zijlstra <a.p.zijlstra@chello.nl>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] perf tools: Fix bison-related build failure on CentOS 6
Message-ID: <20150429191426.GB20445@krava.redhat.com>
References: <1430322871-18107-1-git-send-email-namhyung@kernel.org>
 <20150429162740.GL2079@redhat.com>
 <20150429175705.GB8427@krava.redhat.com>
 <20150429181414.GA14167@krava.redhat.com>
 <20150429184754.GA2200@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20150429184754.GA2200@redhat.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 2267
Lines: 54

On Wed, Apr 29, 2015 at 03:47:54PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Wed, Apr 29, 2015 at 08:14:14PM +0200, Jiri Olsa escreveu:
> > On Wed, Apr 29, 2015 at 07:57:05PM +0200, Jiri Olsa wrote:
> > > On Wed, Apr 29, 2015 at 01:27:40PM -0300, Arnaldo Carvalho de Melo wrote:
> > > > > The YYLTYPE_IS_TRIVIAL is defined in the Build file, but unlike
> > > > > pmu-bison.c, gcc complained about it for parse-events-bison.c:
> > > > > 
> > > > >     CC       util/parse-events-bison.o
> > > > >   In file included from util/parse-events.y:16:
> > > > >   util/parse-events-bison.h:101:1: error: "YYLTYPE_IS_TRIVIAL" redefined
> > > > >   <command-line>: error: this is the location of the previous definition
> > > > >   make[3]: *** [util/parse-events-bison.o] Error 1
> > > > > 
> > > > > Signed-off-by: Namhyung Kim <namhyung@kernel.org>
> > > > 
> > > > Ok, so this helps, it builds now on RHEL6.6, which was something I and
> > > > Jiri were chasing, but why has it complained for one and not the other?
> > > > Ideas?
> > > 
> > > looks like the RHEL6 gcc/cpp treats this as an error which
> > > is not maskable by '-w' option
> > > 
> > > anyway I checked the git log history and I'm not sure
> > > why we event have this define, it was added by Ingo
> > > and carried by ever since:
> > > 
> > >   65f3e56e0c81 perf tools: Remove auto-generated bison/flex files
> > > 
> > > probably because of the fail I see if I remove it from pmu-bison.o build:
> > > 
> > >   util/pmu-bison.c:613:6: error: "YYLTYPE_IS_TRIVIAL" is not defined
> > 
> > ok, seem's the patch is right and the reason is the
> > parse error handling that was added just recently
> > 
> > it adds YYLTYPE type (which is not present in pmu-bison.h),
> > so YYLTYPE_IS_TRIVIAL gets redefined, which is ok in F20
> > that handle the error via '-w' option, but it's not ok for RHEL6
> > where the '-w' does not work for this kind of error
> 
> Ok, adding this comment to the changelog, together with your Acked-by,
> ok?

ook

jirka

> 
> - Arnaldo
--
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/