bin/get-pick-list.sh: use test instead of [ ]
authorEmil Velikov <emil.velikov@collabora.com>
Thu, 8 Nov 2018 15:05:21 +0000 (15:05 +0000)
committerEmil Velikov <emil.l.velikov@gmail.com>
Thu, 15 Nov 2018 15:55:51 +0000 (15:55 +0000)
Latter is rather picky wrt surrounding white space. The explicit `test`
doesn't have that problem, plus the statements read a bit easier.

Cc: mesa-stable@lists.freedesktop.org
Reviewed-by: Juan A. Suarez <jasuarez@igalia.com>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
bin/get-pick-list.sh

index a540cedfbd00452442d0c8f6cf757514e74c3b34..e7fffdd128099031be0e6bfab5d5b5ce629d5d57 100755 (executable)
@@ -30,10 +30,10 @@ is_sha_nomination()
                grep -Eo 'fixes:[a-f0-9]{8,40}'`
 
        fixes_count=`echo "$fixes" | wc -l`
-       if [ $fixes_count -eq 0 ] ; then
+       if test $fixes_count -eq 0; then
                return 0
        fi
-       while [ $fixes_count -gt 0 ] ; do
+       while test $fixes_count -gt 0; do
                # Treat only the current line
                id=`echo "$fixes" | tail -n $fixes_count | head -n 1 | cut -d : -f 2`
                fixes_count=$(($fixes_count-1))
@@ -41,7 +41,7 @@ is_sha_nomination()
                # Bail out if we cannot find suitable id.
                # Any specific validation the $id is valid and not some junk, is
                # implied with the follow up code
-               if [ "x$id" = x ] ; then
+               if test "x$id" = x; then
                        continue
                fi
 
@@ -87,7 +87,7 @@ git log --reverse --pretty=%H -i --grep='^CC:.*mesa-stable\|^CC:.*mesa-dev\|\<fi
 while read sha
 do
        # Check to see whether the patch is on the ignore list.
-       if [ -f bin/.cherry-ignore ] ; then
+       if test -f bin/.cherry-ignore; then
                if grep -q ^$sha bin/.cherry-ignore ; then
                        continue
                fi