From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH] libxl: Fix building libxlu_cfg_y.y with bison 3.0 Date: Wed, 3 Dec 2014 09:53:50 +0000 Message-ID: <1417600430.11243.1.camel@citrix.com> References: <1417325015-22354-1-git-send-email-eswierk@skyportsystems.com> <1417426933.23604.77.camel@citrix.com> <20141201121955.GB19889@zion.uk.xensource.com> <1417528036.24320.32.camel@citrix.com> <547DC617.8020107@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1Xw6dA-0006Og-2Q for xen-devel@lists.xenproject.org; Wed, 03 Dec 2014 09:53:56 +0000 In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ed Swierk Cc: Andrew Cooper , Ian Jackson , Wei Liu , xen-devel@lists.xenproject.org List-Id: xen-devel@lists.xenproject.org On Tue, 2014-12-02 at 09:49 -0800, Ed Swierk wrote: > On Tue, Dec 2, 2014 at 6:00 AM, Andrew Cooper wrote: > > The automatically generating doesn't actually work. Depending on the > > relative timestamps caused by a SCM checkout, or a tarball extraction, > > the files will be attempted to be regenerated. > > > > These files are regenerated in the XenServer build, simply because of > > their order in the archived tarball. > > When I clone the xen tree from git, the timestamps match about 95% of > the time, but the 5% failure rate was annoying enough that I finally > dug in to fix the parser build. > > IMHO the generated files should be omitted from the source tree; as > long as the source files are actually buildable, there's no reason not > to treat them like any other source file. There was a point in time where the prevailing version of bison (or maybe flex) in stable distro releases had a bug which meant these files could not be regenerated easily on common distros. I don't recall the details well enough to know if that time has now passed. Perhaps Ian J does. Ian.