summaryrefslogtreecommitdiffstats
path: root/tools/post-receive.hook
diff options
context:
space:
mode:
authorCarl Worth <cworth@cworth.org>2013-12-11 10:46:27 -0800
committerJeremy Kerr <jk@ozlabs.org>2014-01-29 17:57:12 +0800
commit6dd987e40d5d76dc96ed641723c2ece3f024f85e (patch)
tree05bdaaac33bd120776a3d22b942879b5a490adce /tools/post-receive.hook
parent2b8be9316ca488c8ce913d47dd6f0ad04e7544e8 (diff)
downloadpatchwork-6dd987e40d5d76dc96ed641723c2ece3f024f85e.tar.bz2
patchwork-6dd987e40d5d76dc96ed641723c2ece3f024f85e.tar.xz
Make the post-receive hook more efficient (using pwclient info, not view)
Older versions of patchwork shipped with a version of pwclient that had the view command, but not info command. Now that info exists, it's inefficient to use it to obtain the patchwork ID value, (since it requires the server to send the entire patch content in response to an XML RPC request). So be kind to the server by using info instead, (which has a small, fixed-length response). Signed-off-by: Carl Worth <cworth@cworth.org> Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
Diffstat (limited to 'tools/post-receive.hook')
-rwxr-xr-xtools/post-receive.hook4
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/post-receive.hook b/tools/post-receive.hook
index b2b89a2..4f76fa6 100755
--- a/tools/post-receive.hook
+++ b/tools/post-receive.hook
@@ -25,8 +25,8 @@ get_patchwork_hash()
get_patch_id()
{
local id
- id=$($PWDIR/bin/pwclient view -h $1 2>/dev/null \
- | sed -rne 's,X-Patchwork-Id: ,,p')
+ id=$($PWDIR/bin/pwclient info -h $1 2>/dev/null \
+ | sed -rne 's,- id[[:space:]]*: ,,p')
echo $id
test -n "$id"
}