Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-06-07 Thread Peter Maydell
On 7 June 2013 03:54, Peter Crosthwaite wrote: > Ping! > > We are stuck in limbo here and I would like to get either V1 or V2 > (this patch) of this through. > > Paolo, > > If you are not happy with this patch, does V1 address your concerns > (which was its original intention). Given what David h

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-06-06 Thread Peter Crosthwaite
Ping! We are stuck in limbo here and I would like to get either V1 or V2 (this patch) of this through. Paolo, If you are not happy with this patch, does V1 address your concerns (which was its original intention). Regards, Peter On Sun, Jun 2, 2013 at 9:13 AM, Peter Crosthwaite wrote: > HI Da

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-06-01 Thread Peter Crosthwaite
HI David, On Sat, Jun 1, 2013 at 3:38 PM, David Gibson wrote: > On Fri, May 31, 2013 at 10:44:33AM +0100, Peter Maydell wrote: >> On 31 May 2013 09:25, Paolo Bonzini wrote: >> > Please don't. Fedora is not going to use the bundled dtc because of a >> > policy against bundling, and Fedora's dtc

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-06-01 Thread Peter Crosthwaite
Hi All, On Sat, Jun 1, 2013 at 6:26 PM, Peter Maydell wrote: > On 1 June 2013 06:22, David Gibson wrote: >> On Fri, May 31, 2013 at 10:25:55AM +0200, Paolo Bonzini wrote: >>> Fedora's dtc package doesn't include >>> libfdt_env.h. >> >> Yeah, that makes Fedora's dtc (well, libfdt) package broken.

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-06-01 Thread Peter Maydell
On 1 June 2013 06:22, David Gibson wrote: > On Fri, May 31, 2013 at 10:25:55AM +0200, Paolo Bonzini wrote: >> Fedora's dtc package doesn't include >> libfdt_env.h. > > Yeah, that makes Fedora's dtc (well, libfdt) package broken. > >> If David says that the changes are rare enough, let's just fix t

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-31 Thread David Gibson
On Fri, May 31, 2013 at 10:44:33AM +0100, Peter Maydell wrote: > On 31 May 2013 09:25, Paolo Bonzini wrote: > > Please don't. Fedora is not going to use the bundled dtc because of a > > policy against bundling, and Fedora's dtc package doesn't include > > libfdt_env.h. > > It sounds like Fedora'

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-31 Thread David Gibson
On Fri, May 31, 2013 at 10:25:55AM +0200, Paolo Bonzini wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > Il 31/05/2013 05:32, David Gibson ha scritto: > >>> Then again, although we certainly want to keep changes which > >>> require updates to libfdt_env.h rare, I'm not going to rule >

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-31 Thread Peter Crosthwaite
Hi, On Fri, May 31, 2013 at 7:54 PM, Paolo Bonzini wrote: > Il 31/05/2013 11:44, Peter Maydell ha scritto: >> On 31 May 2013 09:25, Paolo Bonzini wrote: >>> Please don't. Fedora is not going to use the bundled dtc because of a >>> policy against bundling, and Fedora's dtc package doesn't includ

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-31 Thread Paolo Bonzini
Il 31/05/2013 11:44, Peter Maydell ha scritto: > On 31 May 2013 09:25, Paolo Bonzini wrote: >> Please don't. Fedora is not going to use the bundled dtc because of a >> policy against bundling, and Fedora's dtc package doesn't include >> libfdt_env.h. > > It sounds like Fedora's dtc package is br

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-31 Thread Peter Maydell
On 31 May 2013 09:25, Paolo Bonzini wrote: > Please don't. Fedora is not going to use the bundled dtc because of a > policy against bundling, and Fedora's dtc package doesn't include > libfdt_env.h. It sounds like Fedora's dtc package is broken then -- are you going to fix it? > If David says t

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-31 Thread Paolo Bonzini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Il 31/05/2013 05:32, David Gibson ha scritto: >>> Then again, although we certainly want to keep changes which >>> require updates to libfdt_env.h rare, I'm not going to rule >>> out extensions to libfdt which add new (minor) external >>> dependencie

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-30 Thread David Gibson
On Fri, May 31, 2013 at 11:48:47AM +1000, Peter Crosthwaite wrote: > Hi David, > > On Wed, May 29, 2013 at 7:31 PM, David Gibson > wrote: > > On Wed, May 29, 2013 at 10:14:19AM +0100, Peter Maydell wrote: > >> On 29 May 2013 09:18, David Gibson wrote: > >> > On Mon, May 27, 2013 at 02:20:57PM +1

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-30 Thread Peter Crosthwaite
Hi David, On Wed, May 29, 2013 at 7:31 PM, David Gibson wrote: > On Wed, May 29, 2013 at 10:14:19AM +0100, Peter Maydell wrote: >> On 29 May 2013 09:18, David Gibson wrote: >> > On Mon, May 27, 2013 at 02:20:57PM +1000, peter.crosthwa...@xilinx.com >> > wrote: >> >> From: Peter Crosthwaite >>

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-29 Thread David Gibson
On Wed, May 29, 2013 at 10:14:19AM +0100, Peter Maydell wrote: > On 29 May 2013 09:18, David Gibson wrote: > > On Mon, May 27, 2013 at 02:20:57PM +1000, peter.crosthwa...@xilinx.com > > wrote: > >> From: Peter Crosthwaite > >> Currently QEMU provides a local clone of the file libfdt_env.h in > >

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-29 Thread Peter Maydell
On 29 May 2013 09:18, David Gibson wrote: > On Mon, May 27, 2013 at 02:20:57PM +1000, peter.crosthwa...@xilinx.com wrote: >> From: Peter Crosthwaite >> Currently QEMU provides a local clone of the file libfdt_env.h in >> /include. This file is supposed to come with the libfdt package and is > > S

Re: [Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-29 Thread David Gibson
On Mon, May 27, 2013 at 02:20:57PM +1000, peter.crosthwa...@xilinx.com wrote: > From: Peter Crosthwaite > > Currently QEMU provides a local clone of the file libfdt_env.h in > /include. This file is supposed to come with the libfdt package and is So, this patch might be the right thing anyway, b

[Qemu-devel] [PATCH v2 1/1] configure: dtc: Probe for libfdt_env.h

2013-05-26 Thread peter . crosthwaite
From: Peter Crosthwaite Currently QEMU provides a local clone of the file libfdt_env.h in /include. This file is supposed to come with the libfdt package and is only needed for broken installs of libfdt. Now that we have submodule dtc, just ignore these broken installs and prompt for the dtc subm