[PATCH] templates: Add click-to-copy patch ID ("mpe mode") to patch detail page

2017-12-19 Thread Andrew Donnellan
Similar to what we already do on the patch list page, display the patch ID on the patch detail page and make it a click-to-copy button. Closes: #115 ("Show copy-able patch ID ("mpe mode") on patch detail page") Signed-off-by: Andrew Donnellan --- patchwork/templates/patchwork/download_buttons.ht

Re: [PATCH v3 5/5] tools: Update to use 'hasher'

2017-12-19 Thread Tom Rini
On Thu, Dec 01, 2016 at 09:46:52AM +, Stephen Finucane wrote: > The old 'parser' module used to extract diffs from their surrounding > mbox fluff before hashing this. Seeing as this was only used in the > context of an actual git repo, avoid all of that rigmarole by just using > 'git diff', whi

[PATCH] post-receive.hook: Handle failure to find patch number

2017-12-19 Thread Tom Rini
When pwclient info -h fails to come up with the number for the change in question it will exit with a non-zero exit code. This failure will propagate upwards and exit the script there. Make our call to get_patch_id or in true so that our script here will see an empty id and we continue on with th

Re: [PATCH v3 3/5] hasher: Create hasher module

2017-12-19 Thread Tom Rini
On Thu, Dec 01, 2016 at 09:46:50AM +, Stephen Finucane wrote: > This exposes the hashing functionality of Patchwork without requiring > Django or similar dependencies. > > Signed-off-by: Stephen Finucane > Cc: Paul Jakma > Cc: Tom Rini Tested-by: Tom Rini -- Tom signature.asc Descrip

Re: [PATCH v3 4/5] tools: Trivial formatting fixes

2017-12-19 Thread Tom Rini
On Thu, Dec 01, 2016 at 09:46:51AM +, Stephen Finucane wrote: > These tools are currently broken, but before beginning surgery let's > clean things up. Use standard 4 spaces and the longer, but easier to > read, if-else-fi syntax for comparison. > > Missing license headers are added for compl

Re: [PATCH v3 2/5] parser: Trivial rename of functions

2017-12-19 Thread Tom Rini
On Thu, Dec 01, 2016 at 09:46:49AM +, Stephen Finucane wrote: > 'auto_delegate' doesn't actually delegate the patches - it merely finds > a suitable delegate based on the filename. Rename the function > accordingly. > > 'find_delegate' is also renamed to prevent confusion. > > Signed-off-by:

Re: [PATCH v3 1/5] parser: Remove unused parameter

2017-12-19 Thread Tom Rini
On Thu, Dec 01, 2016 at 09:46:48AM +, Stephen Finucane wrote: > The find_content function expected a 'project' parameter but never > actually used it. Remove it and clean up tests accordingly. > > Signed-off-by: Stephen Finucane > Cc: Paul Jakma > Cc: Tom Rini Tested-by: Tom Rini -- To

Re: [5/5] tools: Update to use 'parsemail --hash'

2017-12-19 Thread Tom Rini
On Thu, Dec 01, 2016 at 01:21:40PM +, Stephen Finucane wrote: > On Tue, 2016-11-29 at 18:53 -0500, Tom Rini wrote: > > On Fri, Nov 18, 2016 at 12:54:52AM +, Stephen Finucane wrote: > > > > > This replaces the older 'parser' main function. > > > > > > Signed-off-by: Stephen Finucane > > >

Re: [5/5] tools: Update to use 'parsemail --hash'

2017-12-19 Thread Tom Rini
On Fri, Nov 18, 2016 at 12:54:52AM +, Stephen Finucane wrote: > This replaces the older 'parser' main function. > > Signed-off-by: Stephen Finucane > Cc: Paul Jakma With this series on top of 2941d6f47fd242d53c52e966501fb09e2eb19473 I don't have success, but instead: Traceback (most recent

Re: Patchwork v2.0.0 Available

2017-12-19 Thread Aaron Conole
David Miller writes: > From: Michael Ellerman > Date: Mon, 28 Aug 2017 20:12:49 +1000 > >> Jeremy Kerr writes: >> >>> Hi all, >>> Just an update: the database migration for v2.0.0 is taking a *long* time, so we'll be out for more than that one hour. I'll get things back up and r