Patchwatcher status

[Dan Kegel]

Patchwatcher falsely complained that
 [2/17] richedit: Removed assumption about the order of rtf indent
didn't apply because the regexp I used to detect
the end of a patch series falsely matched the
first patch in a series of 1x patches.  Here's the fix:

--- patchwatcher.sh     (revision 150)
+++ patchwatcher.sh     (working copy)
@@ -283,7 +283,7 @@
            cat $PATCHES/$NEXT.log
         fi
         # Use a regexp with a back reference to detect last patch in
a series and break out
-        if egrep -q 'Subject:.*[0-9]+/[0-9]+' $PATCHES/$NEXT.txt && !
egrep -q 'Subject:.*([0-9]+)/\1' $PATCHES/$NEXT.txt
+        if egrep -q 'Subject:.*[0-9]+/[0-9]+' $PATCHES/$NEXT.txt && !
egrep -q 'Subject:.*([0-9]+)/\1[^0-9]' $PATCHES/$NEXT.txt
         then
             echo In middle of patch series, not wiping tree
             NEXT=`expr $NEXT + 1`

And it falsely complained that various of adam at tpetaccia.com's
patches failed because it didn't handle the case of multiple
patch series sent by the same author back to back and mixed
together by email delays.  Here's a fix:

--- get-patches.pl      (revision 150)
+++ get-patches.pl      (working copy)
@@ -147,7 +147,7 @@
        $series_num_patches = $num_patches;
     }

-    if ($series_sender ne $sender) {
+    if ($series_sender ne $sender || $series_num_patches != $num_patches) {
         #print "Ignoring series for now, will try later; sender
$sender, num_patches $num_patches, subject
".$header->get('Subject')."\n";
         # can't handle multiple series at once just yet, let it sit
         return;

That still won't handle the case of two patch series of the same length
being sent back to back by the same author, but that's
ambiguous enough that even humans might be confused,
so it should tide us over until I rewrite the patch series
detector to handle incomplete series better.

I'm on vacation so I can't apply these patches to the running instance,
I'll do it on tuesday.
- Dan