From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MlnIs-0006dB-Js for qemu-devel@nongnu.org; Thu, 10 Sep 2009 13:19:26 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MlnIn-0006cn-7p for qemu-devel@nongnu.org; Thu, 10 Sep 2009 13:19:25 -0400 Received: from [199.232.76.173] (port=53644 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MlnIm-0006cj-VH for qemu-devel@nongnu.org; Thu, 10 Sep 2009 13:19:21 -0400 Received: from mail.gmx.net ([213.165.64.20]:40681) by monty-python.gnu.org with smtp (Exim 4.60) (envelope-from ) id 1MlnIm-0006n8-Fp for qemu-devel@nongnu.org; Thu, 10 Sep 2009 13:19:20 -0400 Date: Thu, 10 Sep 2009 19:19:03 +0200 From: Reimar =?iso-8859-1?Q?D=F6ffinger?= Subject: Re: [Qemu-devel] [PATCH] [RESEND2] Qemu unmaintained? Message-ID: <20090910171903.GA16143@1und1.de> References: <4AA7A6EC.10907@codemonkey.ws> <20090910070336.GD3351@amit-x200.redhat.com> <4AA90592.7080100@codemonkey.ws> <4AA90F7F.2030709@redhat.com> <4AA92122.3050103@codemonkey.ws> <4AA924AE.8060807@redhat.com> <4AA927D8.7000900@codemonkey.ws> <4AA92ADF.80003@redhat.com> <4AA92B7A.7010304@codemonkey.ws> <4AA92D5F.9070205@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <4AA92D5F.9070205@redhat.com> List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Avi Kivity Cc: Amit Shah , Mark McLoughlin , Bernhard Kauer , qemu-devel@nongnu.org On Thu, Sep 10, 2009 at 07:46:23PM +0300, Avi Kivity wrote: > On 09/10/2009 07:38 PM, Anthony Liguori wrote: > > Well the question is, should I/you edit this, or reject the patch > > requesting a better changelog? > > > > Certainly, the later is what akpm often does. > > I'm happy to reject patches for whitespace but I will edit changelogs. > My rationale is that many people don't care about that and I can't make > them care; further the log is mostly for my own benefit - I spend quite > a lot of time reading it when hunting regressions or preparing > patchqueues for upstream. I'd also add that anyone used to projects using SVN will probably be used to writing only a general explanation of the patch while the real commit message is left to whoever commits it. Maybe they ideally should be identical, but I expect that quite a few people _won't_ expect their email to appear 1:1 in the repository as log message (I usually feel quite embarrassed when my hastily written email by which I only wanted to get first comments ends up in a project's permanent history). I am sure it misses a lot of stuff and there might even be objections to some parts, but I wrote this draft of a "PATCHES" (or "CONTRIBUTING" or whatever) file that might help newcomers (and even some long-term developers might not know all the unwritten rules ;-). Suggested text: This is a (very) rough guide on how to submit patches to qemu, what is expected of you and what to expect from the process. Patches should go to the qemu-devel@nongnu.org list, subscription is possible via http://lists.nongnu.org/mailman/listinfo/qemu-devel The subject for any emails containing patches should start with [PATCH] so it is obvious that there is a patch included. Whenever you send a new patch or a new version of a patch, you should start a new thread - i.e. do _not_ reply to any email but write a new one. Patches are preferred inline (i.e. not as attachments - but be careful your mailer does not mangle them e.g. by adding line breaks). Emails generated via "git format-patch" are even better. Also be aware that emails are often used as-is as log messages, so try to write your emails in a way that is suitable for this, in particular they should in an understandable and not too verbose way describe what change was made and why. Also do not forget to add the appropriate Signed-off-by lines. Do not expect an immediate reply to your patches, reacting to patches simply takes some time. If there is no reaction and you checked that the patch was not already applied (there currently is no notification about this) try sending the patch once again, it might just have got lost or forgotten at some point.