From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39864) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vl3jI-0001CF-Gp for qemu-devel@nongnu.org; Mon, 25 Nov 2013 16:30:10 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Vl3jB-0008L8-Fu for qemu-devel@nongnu.org; Mon, 25 Nov 2013 16:30:04 -0500 Received: from mx1.redhat.com ([209.132.183.28]:60423) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vl3jB-0008Ky-8F for qemu-devel@nongnu.org; Mon, 25 Nov 2013 16:29:57 -0500 Received: from int-mx12.intmail.prod.int.phx2.redhat.com (int-mx12.intmail.prod.int.phx2.redhat.com [10.5.11.25]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id rAPLTu8t010375 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Mon, 25 Nov 2013 16:29:56 -0500 Date: Mon, 25 Nov 2013 23:33:12 +0200 From: "Michael S. Tsirkin" Message-ID: <20131125213312.GB13692@redhat.com> References: <1385379990-32093-1-git-send-email-mst@redhat.com> <1385379990-32093-4-git-send-email-mst@redhat.com> <5293B215.4030108@redhat.com> <20131125203152.GB12689@redhat.com> <5293B517.8090108@redhat.com> <20131125210247.GK12689@redhat.com> <5293BF70.3030704@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5293BF70.3030704@redhat.com> Subject: Re: [Qemu-devel] [PULL for-1.7 v2 3/6] acpi-build: fix build on glib < 2.22 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Richard Henderson Cc: Paolo Bonzini , qemu-devel@nongnu.org On Tue, Nov 26, 2013 at 07:21:52AM +1000, Richard Henderson wrote: > On 11/26/2013 07:02 AM, Michael S. Tsirkin wrote: > > In any case it's too late to play with cosmetic changes for 1.7. > > We can revisit for 1.8. > > I beg your pardon? I didn't realize your patch had already been applied. When you see PATCH you should review. When you see PULL it's been applied and pushed to a public branch. > And > since it has mistakes that require respin anyway, why can't we change this too? > > > r~ The patch we discuss is correct, the patch removing get element size didn't remove it in all places, but it's not making things any worse so I'll fix them with applying a patch on top. I queued other things on top and it's 11:30pm here. Guys, go discuss the color of some other bikeshed. -- MST