diff mbox

[2/3] Make the post-receive hook more efficient (using pwclient info, not view)

Message ID 1386787588-2074-2-git-send-email-cworth@cworth.org
State Accepted
Headers show

Commit Message

Carl Worth Dec. 11, 2013, 6:46 p.m. UTC 
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).
---
 tools/post-receive.hook | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff mbox

Patch

diff --git a/tools/post-receive.hook b/tools/post-receive.hook
index b2b89a2..d0b8e22 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 -p mesa -h $1 2>/dev/null \
+    | sed -rne 's,- id[[:space:]]*: ,,p')
   echo $id
   test -n "$id"
 }