From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53541) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V0Y5s-00081R-23 for qemu-devel@nongnu.org; Sat, 20 Jul 2013 10:25:09 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V0Y5q-0002wE-Nz for qemu-devel@nongnu.org; Sat, 20 Jul 2013 10:25:07 -0400 Message-ID: <51EA9DBA.4080400@suse.de> Date: Sat, 20 Jul 2013 16:24:58 +0200 From: =?ISO-8859-1?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <1373647719-32757-1-git-send-email-sw@weilnetz.de> <51E4E011.70407@weilnetz.de> <51E846AC.5070309@weilnetz.de> <51E98C93.8010508@weilnetz.de> In-Reply-To: <51E98C93.8010508@weilnetz.de> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH] PPC: dbdma: macio: Fix format specifiers (build regression) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Weil , Alexander Graf Cc: qemu-trivial , Anthony Liguori , qemu-devel Am 19.07.2013 20:59, schrieb Stefan Weil: > Am 19.07.2013 00:40, schrieb Alexander Graf: >> On 18.07.2013, at 21:49, Stefan Weil wrote: >>> Am 16.07.2013 07:54, schrieb Stefan Weil: >>>> Am 12.07.2013 18:48, schrieb Stefan Weil: >>>>> Hi Anthony, >>>>> >>>>> the patch fixes a build regression which was introduced today. >>>>> Could you please apply it without waiting for the next pull request= s? >>>>> >>>>> Thanks, >>>>> Stefan > [...] >> Acked-by: Alexander Graf I assume this can go through >> the trivial tree? Or directly get applied by Anthony? Alex=20 >=20 >=20 > I still think that build regressions should be fixed by a direct > maintainer commit instead of waiting for a pull request, > but obviously this does not happen here (why?). Anthony recently stated very clearly that in order for him to pick up a patch it needs to have a Reviewed-by. This one only got an Acked-by. > So hopefully the fix will be committed via qemu-trivial at least. It's not obvious from the diff context whether those specifiers are in fact correct, and there is an active maintainer, so in theory it wouldn't qualify for -trivial. Alex, don't be lazy and just send a one-patch PULL like Cornelia did for her recent mingw breakage. :) Andreas > See http://patchwork.ozlabs.org/patch/258774/ for the full patch. >=20 > Stefan --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 16746 AG N=FCrnbe= rg