[PATCH v2 5/6] tools: Trivial formatting fixes

Stephen Finucane stephen at that.guru
Thu Dec 1 05:42:46 AEDT 2016


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 completeness sake.

Signed-off-by: Stephen Finucane <stephen at that.guru>
Cc: Paul Jakma <paul at jakma.org>
Cc: Tom Rini <trini at konsulko.com>
---
 tools/patchwork-update-commits |  33 +++++++++---
 tools/post-receive.hook        | 116 ++++++++++++++++++++++++-----------------
 2 files changed, 94 insertions(+), 55 deletions(-)

diff --git a/tools/patchwork-update-commits b/tools/patchwork-update-commits
index 820fd1c..d0f63a9 100755
--- a/tools/patchwork-update-commits
+++ b/tools/patchwork-update-commits
@@ -1,17 +1,34 @@
 #!/bin/bash
+#
+# Patchwork - automated patch tracking system
+# Copyright (C) 2010 Jeremy Kerr <jk at ozlabs.org>
+#
+# This file is part of the Patchwork package.
+#
+# Patchwork is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# Patchwork is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with Patchwork; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
 toolsdir="$(dirname "$0")"
 pwpath="${toolsdir}"/../patchwork
 
-if [ "$#" -lt 1 ]
-then
-	echo "usage: $0 <revspec>" >&2
-	exit 1
+if [ "$#" -lt 1 ]; then
+    echo "usage: $0 <revspec>" >&2
+    exit 1
 fi
 
 git rev-list --reverse "$@" |
-while read commit
-do
-	hash=$(git show "$commit" | python $pwpath/parser.py -#)
-	$pwpath/bin/pwclient update -s Accepted -c "$commit" -h "$hash"
+while read commit; do
+    hash=$(git show "$commit" | python $pwpath/parser.py -#)
+    $pwpath/bin/pwclient update -s Accepted -c "$commit" -h "$hash"
 done
diff --git a/tools/post-receive.hook b/tools/post-receive.hook
index a4e8405..a19e1b2 100755
--- a/tools/post-receive.hook
+++ b/tools/post-receive.hook
@@ -1,9 +1,24 @@
 #!/bin/bash
+# Patchwork - automated patch tracking system
+# Copyright (C) 2010 martin f. krafft <madduck at madduck.net>
 #
-# Git post-receive hook to update Patchwork patches after Git pushes
+# This file is part of the Patchwork package.
+#
+# Patchwork is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# Patchwork is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
 #
-# Copyright © 2010 martin f. krafft <madduck at madduck.net>
-# Released under the GNU General Public License v2 or later.
+# You should have received a copy of the GNU General Public License
+# along with Patchwork; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+
+# Git post-receive hook to update Patchwork patches after Git pushes
 set -eu
 
 #TODO: the state map should really live in the repo's git-config
@@ -22,58 +37,65 @@ trap "do_exit=1" INT
 
 get_patchwork_hash()
 {
-  local hash
-  hash=$(git show -C $1 | python $PWDIR/parser.py --hash)
-  echo $hash
-  test -n "$hash"
+    local hash
+    hash=$(git show -C $1 | python $PWDIR/parser.py --hash)
+    echo $hash
+    test -n "$hash"
 }
 
-get_patch_id()
-{
-  local id
-  id=$($PWDIR/bin/pwclient info -h $1 2>/dev/null \
-    | sed -rne 's,- id[[:space:]]*: ,,p')
-  echo $id
-  test -n "$id"
+get_patch_id() {
+    local id
+    id=$($PWDIR/bin/pwclient info -h $1 2>/dev/null \
+         | sed -rne 's,- id[[:space:]]*: ,,p')
+    echo $id
+    test -n "$id"
 }
 
-set_patch_state()
-{
-  $PWDIR/bin/pwclient update -s $2 -c $3 $1 2>&1
+set_patch_state() {
+    $PWDIR/bin/pwclient update -s $2 -c $3 $1 2>&1
 }
 
-update_patches()
-{
-  local cnt; cnt=0
-  for rev in $(git rev-parse --not ${EXCLUDE} |
-               git rev-list --stdin --no-merges --reverse ${1}..${2}); do
-    if [ "$do_exit" = 1 ]; then
-      echo "I: exiting..." >&2
-      break
-    fi
-    hash=$(get_patchwork_hash $rev) \
-      || { echo "E: failed to hash rev $rev." >&2; continue; }
-    id=$(get_patch_id $hash) \
-      || { echo "E: failed to find patch for rev $rev." >&2; continue; }
-    reason="$(set_patch_state $id $3 $rev)" \
-      || { echo "E: failed to update patch #$id${reason:+: $reason}." >&2; continue; }
-    echo "I: patch #$id updated using rev $rev." >&2
-    cnt=$(($cnt + 1))
-  done
-  echo "I: $cnt patch(es) updated to state $3." >&2
+update_patches() {
+    local cnt; cnt=0
+    for rev in $(git rev-parse --not ${EXCLUDE} |
+                 git rev-list --stdin --no-merges --reverse ${1}..${2}); do
+        if [ "$do_exit" = 1 ]; then
+            echo "I: exiting..." >&2
+            break
+        fi
+        hash=$(get_patchwork_hash $rev)
+        if [ -z "$hash" ]; then
+            echo "E: failed to hash rev $rev." >&2
+            continue
+        fi
+        id=$(get_patch_id $hash)
+        if [ -z "$id" ]; then
+            echo "E: failed to find patch for rev $rev." >&2
+            continue
+        fi
+        reason="$(set_patch_state $id $3 $rev)"
+        if [ -z "$reason" ]; then
+            echo "E: failed to update patch #$id${reason:+: $reason}." >&2
+            continue
+        fi
+        echo "I: patch #$id updated using rev $rev." >&2
+        cnt=$(($cnt + 1))
+    done
+
+    echo "I: $cnt patch(es) updated to state $3." >&2
 }
 
 while read oldrev newrev refname; do
-  found=0
-  for i in $STATE_MAP; do
-    key="${i%:*}"
-    if [ "$key" = "$refname" ]; then
-      update_patches $oldrev $newrev ${i#*:}
-      found=1
-      break
+    found=0
+    for i in $STATE_MAP; do
+        key="${i%:*}"
+        if [ "$key" = "$refname" ]; then
+            update_patches $oldrev $newrev ${i#*:}
+            found=1
+            break
+        fi
+    done
+    if [ $found -eq 0 ]; then
+        echo "E: STATE_MAP has no mapping for branch $refname" >&2
     fi
-  done
-  if [ $found -eq 0 ]; then
-    echo "E: STATE_MAP has no mapping for branch $refname" >&2
-  fi
 done
-- 
2.9.3



More information about the Patchwork mailing list