From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:52302)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <edgar.iglesias@gmail.com>) id 1USnUA-0000SN-8U
	for qemu-devel@nongnu.org; Thu, 18 Apr 2013 07:58:43 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <edgar.iglesias@gmail.com>) id 1USnU8-0001Vh-5g
	for qemu-devel@nongnu.org; Thu, 18 Apr 2013 07:58:42 -0400
Received: from mail-ea0-x22d.google.com ([2a00:1450:4013:c01::22d]:53708)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <edgar.iglesias@gmail.com>) id 1USnU7-0001VU-VG
	for qemu-devel@nongnu.org; Thu, 18 Apr 2013 07:58:40 -0400
Received: by mail-ea0-f173.google.com with SMTP id k11so1189815eaj.4
	for <qemu-devel@nongnu.org>; Thu, 18 Apr 2013 04:58:39 -0700 (PDT)
Date: Thu, 18 Apr 2013 13:58:34 +0200
From: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
Message-ID: <20130418115833.GJ4915@smtp.vpn>
References: <cover.1366260005.git.peter.crosthwaite@xilinx.com>
	<20130418102619.GG4915@smtp.vpn> <516FD2D1.1060403@redhat.com>
	<20130418110854.GH4915@smtp.vpn>
	<CAFEAcA8xVX7xeExhJ579XmgyLff=8dX6WOTu39wbjZJfefnZFg@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAFEAcA8xVX7xeExhJ579XmgyLff=8dX6WOTu39wbjZJfefnZFg@mail.gmail.com>
Subject: Re: [Qemu-devel] [PATCH v5 0/3] DTC as submodule
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Paolo Bonzini <pbonzini@redhat.com>, peter.crosthwaite@xilinx.com, qemu-devel@nongnu.org, aliguori@us.ibm.com

On Thu, Apr 18, 2013 at 12:10:54PM +0100, Peter Maydell wrote:
> On 18 April 2013 12:08, Edgar E. Iglesias <edgar.iglesias@gmail.com> wrote:
> > On Thu, Apr 18, 2013 at 01:02:41PM +0200, Paolo Bonzini wrote:
> >> For submodules that do have autoconf-generated scripts in the repository
> >> (I think both of them do, in our case), we should trust upstream's
> >> configure script.  That's what everyone has been testing the git
> >> repository with.
> >
> > I downloaded the qemu-1.4.1 release and looked at the pixman subdir. It
> > doesnt come with a generated configure script, but has a autogen.sh script to
> > run autoconf itself. My understanding was that, like you say, the user
> > shouldn't need to run autoconf to generate configure, it should have been
> > provided. Am I missing something?
> 
> Our release notes from when we put pixman in:
> http://wiki.qemu.org/ChangeLog/1.3#Build_dependencies
> say that you need autoconf if you're using the built in submodule.
> I think that's just documentation of the current situation rather
> than an explicit decision that it's the right approach, though.

Thanks, that explains it.

Cheers,
Edgar