From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:50482) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SzGla-0007RG-GC for qemu-devel@nongnu.org; Wed, 08 Aug 2012 20:38:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SzGlZ-0006q4-Hs for qemu-devel@nongnu.org; Wed, 08 Aug 2012 20:38:22 -0400 Received: from mx1.redhat.com ([209.132.183.28]:32520) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SzGlZ-0006pw-8A for qemu-devel@nongnu.org; Wed, 08 Aug 2012 20:38:21 -0400 Date: Wed, 8 Aug 2012 21:38:47 -0300 From: Luiz Capitulino Message-ID: <20120808213847.504d911f@doriath.home> In-Reply-To: <5022D7A9.6020908@linux.vnet.ibm.com> References: <1344355108-14786-1-git-send-email-coreyb@linux.vnet.ibm.com> <1344355108-14786-3-git-send-email-coreyb@linux.vnet.ibm.com> <5021472E.3090501@redhat.com> <50214B5B.5030008@linux.vnet.ibm.com> <502193B5.30304@redhat.com> <5022B8D6.9070706@linux.vnet.ibm.com> <20120808174824.7565f531@doriath.home> <5022D199.1070402@linux.vnet.ibm.com> <20120808181300.0eef737e@doriath.home> <5022D7A9.6020908@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v7 2/6] qapi: Introduce add-fd, remove-fd, query-fdsets List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Corey Bryant Cc: kwolf@redhat.com, aliguori@us.ibm.com, stefanha@linux.vnet.ibm.com, libvir-list@redhat.com, qemu-devel@nongnu.org, Eric Blake On Wed, 08 Aug 2012 17:18:33 -0400 Corey Bryant wrote: > > > On 08/08/2012 05:13 PM, Luiz Capitulino wrote: > > On Wed, 08 Aug 2012 16:52:41 -0400 > > Corey Bryant wrote: > > > >> > >> > >> On 08/08/2012 04:48 PM, Luiz Capitulino wrote: > >>> On Wed, 08 Aug 2012 15:07:02 -0400 > >>> Corey Bryant wrote: > >>> > >>>> > >>>> > >>>> On 08/07/2012 06:16 PM, Eric Blake wrote: > >>>>> On 08/07/2012 11:07 AM, Corey Bryant wrote: > >>>>> > >>>>>>>> +# > >>>>>>>> +# Since: 1.2.0 > >>>>>>> > >>>>>>> We're not very consistent on '1.2' vs. '1.2.0' in since listings, but > >>>>>>> that's probably worth a global cleanup closer to hard freeze. > >>>>>>> > >>>>>> > >>>>>> I'll make a note of it. Or does Luiz usually do a cleanup? > >>>>> > >>>>> No idea. > >>>>> > >>>> > >>>> Luiz, were you planning to take a pass at cleaning up the "since" > >>>> release? If not let me know and I can submit a patch. Just let me know > >>>> which to use, '1.2' or '1.2.0'. > >>> > >>> I'd appreciate it if you submit a patch. I guess we should use 1.2.0. > >>> > >> > >> Sure I'll do that. > >> > >> Btw, should I be using error_set(errp, ERROR_CLASS_GENERIC_ERROR, "...") > >> and have a dependency on your patch series? Or should I stick with the > >> old error messages? > > > > That would preferable, yes. Specially if you're adding new errors. > > > > But I'm not exactly sure when my series will be merged, this depends on > > what review will bring. > > > > Are you planning on getting in QEMU 1.2? Absolutely.