From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MvH3J-0003KN-DQ for qemu-devel@nongnu.org; Tue, 06 Oct 2009 16:54:33 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MvH3D-0003Eh-NW for qemu-devel@nongnu.org; Tue, 06 Oct 2009 16:54:32 -0400 Received: from [199.232.76.173] (port=45768 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MvH3D-0003De-1D for qemu-devel@nongnu.org; Tue, 06 Oct 2009 16:54:27 -0400 Received: from mail.gmx.net ([213.165.64.20]:34809) by monty-python.gnu.org with smtp (Exim 4.60) (envelope-from ) id 1MvH3C-0005GU-Ci for qemu-devel@nongnu.org; Tue, 06 Oct 2009 16:54:26 -0400 Message-ID: <4ACBAE7F.4010803@gmx.net> Date: Tue, 06 Oct 2009 22:54:23 +0200 From: Carl-Daniel Hailfinger MIME-Version: 1.0 Subject: Re: [Qemu-devel] patchwork service. References: <200909142331.49181.jcd@tribudubois.net> <4AAEC8CC.8030602@codemonkey.ws> <4AAEE220.8040706@gmx.net> <20091006163845.GA3548@redhat.com> In-Reply-To: <20091006163845.GA3548@redhat.com> Content-Type: multipart/mixed; boundary="------------050803000000080609020605" List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: patchwork@lists.ozlabs.org, qemu-devel@nongnu.org, Jean-Christophe Dubois This is a multi-part message in MIME format. --------------050803000000080609020605 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit [Adding the patchwork mailing list to CC:] On 06.10.2009 18:38, Michael S. Tsirkin wrote: > On Tue, Sep 15, 2009 at 02:38:56AM +0200, Carl-Daniel Hailfinger wrote: > >> On 15.09.2009 00:50, Anthony Liguori wrote: >> >>> Jean-Christophe Dubois wrote: >>> >>>> http://patchwork.ozlabs.org/project/qemu-devel/list/ >>>> >>>> But it is not used as the patch status is not updated for quite a few >>>> of them (even if they have been accepted/merged since then). >>>> >>> I looked at the patchwork tooling and it didn't seem to easily >>> integrate into my work flow. It seems to be designed with the idea >>> that you use their tools to pull and manage patches. I don't think >>> those tools will scale to the volume of patches qemu-devel sees >>> though. Plus, there's no easy way to deal with the fact that other >>> maintainers may not use it. Now there are a few thousand patches and >>> trolling through would take ages. >>> >> Note that there are scripts which can be placed in commit hooks and >> automatically mark patches as applied if they were not changed between >> submission and applying. >> There are also scripts which walk the commit history between commit A >> and commit B and run the commit hook script for every commit in between. >> > > Could you post a link to these scripts please? > The scripts I saw were git-based, but I can't remember where I found them. Maybe someone on the patchwork list knows. I have attached my personal script which is centered around flashrom and its svn repo. Please note that the two svn invocations are intentional. The first one retrieves a patch designed to be applied with patch -p0, the second one retrieves the same patch, but from a higher directory so you have to apply with -p1. AFAIK patchwork generates a hash which normalizes the first component of the path if it is a directory, so the hash will be identical for manual diffs from a level above the base directory and git diffs, but manual diffs inside the base directory will have a different hash. Generating both styles of patches from the repo increases matching reliability. You will have to change the lines which calculate hashcand to retrieve the patches from git instead. The script expects a correct ~/.pwclientrc with your password. For flashrom, I just run # update-patchwork 743 if I want to mark revision 743 as committed. You can look at the corresponding patchwork page here: http://patchwork.coreboot.org/patch/323/ . Note "State Accepted" and "Commit r743" at the beginning of the page. Regards, Carl-Daniel -- http://www.hailfinger.org/ --------------050803000000080609020605 Content-Type: text/plain; name="update-patchwork" Content-Transfer-Encoding: base64 Content-Disposition: inline; filename="update-patchwork" IyEvYmluL2Jhc2gKdGVzdCAteiAiJDEiICYmIHsgZWNobyAiTm8gcmV2aXNpb24gcHJvdmlk ZWQuIjsgZXhpdCAxOyB9CnJldmlzaW9uPSIkMSIKaGFzaGNhbmQ9JChzdm4gZGlmZiAtYyAk cmV2aXNpb24gc3ZuOi8vY29yZWJvb3Qub3JnL2ZsYXNocm9tL3RydW5rL3xweXRob24gL3Nv dXJjZXMvdG1wdHJlZXMvcGF0Y2h3b3JrL2FwcHMvcGF0Y2h3b3JrL3BhcnNlci5weSAtLWhh c2gpCi9zb3VyY2VzL3RtcHRyZWVzL3BhdGNod29yay9hcHBzL3BhdGNod29yay9iaW4vcHdj bGllbnQgdXBkYXRlIC1zIEFjY2VwdGVkIC1jIHIkcmV2aXNpb24gLWggJGhhc2hjYW5kICYm IHsgZWNobyAiZG9uZSI7IGV4aXQgMDsgfQplY2hvCmhhc2hjYW5kPSQoc3ZuIGRpZmYgLWMg JHJldmlzaW9uIHN2bjovL2NvcmVib290Lm9yZy9mbGFzaHJvbS98cHl0aG9uIC9zb3VyY2Vz L3RtcHRyZWVzL3BhdGNod29yay9hcHBzL3BhdGNod29yay9wYXJzZXIucHkgLS1oYXNoKQov c291cmNlcy90bXB0cmVlcy9wYXRjaHdvcmsvYXBwcy9wYXRjaHdvcmsvYmluL3B3Y2xpZW50 IHVwZGF0ZSAtcyBBY2NlcHRlZCAtYyByJHJldmlzaW9uIC1oICRoYXNoY2FuZCAmJiB7IGVj aG8gImRvbmUiOyBleGl0IDA7IH0KZWNobyAiTm8gbWF0Y2hpbmcgcGF0Y2ggaW4gcGF0Y2h3 b3JrIGZvdW5kLiIKZXhpdCAxCg== --------------050803000000080609020605--