From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:33010)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <anthony@codemonkey.ws>) id 1SQ6aa-0007lj-TT
	for qemu-devel@nongnu.org; Thu, 03 May 2012 20:41:42 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <anthony@codemonkey.ws>) id 1SQ6aY-0004KY-Vv
	for qemu-devel@nongnu.org; Thu, 03 May 2012 20:41:40 -0400
Received: from mail-ob0-f173.google.com ([209.85.214.173]:48369)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <anthony@codemonkey.ws>) id 1SQ6aY-0004KL-R1
	for qemu-devel@nongnu.org; Thu, 03 May 2012 20:41:38 -0400
Received: by obbwd20 with SMTP id wd20so3671643obb.4
	for <qemu-devel@nongnu.org>; Thu, 03 May 2012 17:41:37 -0700 (PDT)
Message-ID: <4FA325BD.40201@codemonkey.ws>
Date: Thu, 03 May 2012 19:41:33 -0500
From: Anthony Liguori <anthony@codemonkey.ws>
MIME-Version: 1.0
References: <CAFEAcA_a2Cr5nrC3DtSc+X7gXYtMw1TTG+Qm_g4Tk409zLQCVA@mail.gmail.com>
In-Reply-To: <CAFEAcA_a2Cr5nrC3DtSc+X7gXYtMw1TTG+Qm_g4Tk409zLQCVA@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] unreviewed commits (was: Re: Restore consistent
 formatting)
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: qemu-devel@nongnu.org, Markus Armbruster <armbru@redhat.com>

On 05/03/2012 02:58 PM, Peter Maydell wrote:
> On 9 February 2012 13:46, Anthony Liguori<anthony@codemonkey.ws>  wrote:
>> On 02/09/2012 03:48 AM, Markus Armbruster wrote:
>>> You buried the one truly important sentence, let me dig it out for you:
>>>
>>>          *** Patches should always go to the mailing list ***
>>>
>>> Exceptions need justification.  Responsible handling embargoed security
>>> issues may qualify.  Style fixes certainly not.
>>
>> 100% agreed.
>
> I don't see anything in the mailing list archives corresponding
> to commits f05ae537, f6af014e.
>
> No unreviewed patches should go double when we're in hardfreeze!

These patches are admittedly trivial but it is important to stress the point 
that all patches need to go on the mailing list before being committed.

It's an important part of keeping the development process inclusive.  I don't 
think it's reasonable to ask for an Acked-by on something as simple as 
indentation changes but at the same time, there's no reason not to just post 
patches.

Regards,

Anthony Liguori

>
> -- PMM
>