Dan Kegel wrote: > 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 [EMAIL PROTECTED]'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 > > Hi Dan,
Looks like there is something (else?) wrong with checking the patch series. None of the last patch series (by Juan, James, Rob, Stefan to name a few) are shown on the results page. Cheers, Paul.