<div dir="ltr">Hi there Petri,<div><br></div><div>I apologize for my ignorance. This is my first time contributing over</div><div>mailing list. Thank you for being understanding. :)</div><div><br></div><div>I squashed all three patches into one and sent it under a new thread.</div><div>Hopefully that should make things easier for Patchwork.</div><div><br></div><div>Jake Freeland</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 17, 2022 at 10:17 AM Petri Latvala <<a href="mailto:petri.latvala@intel.com">petri.latvala@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, Aug 17, 2022 at 09:30:22AM -0500, Jake Freeland wrote:<br>
> It appears that Patchwork is not picking up the second patch in revision 2:<br>
> <a href="https://patchwork.freedesktop.org/series/107314/#rev2" rel="noreferrer" target="_blank">https://patchwork.freedesktop.org/series/107314/#rev2</a><br>
> <br>
> Is there a way I can manually add the second patch?<br>
<br>
You replied to patch 2, making patchwork think it's another version of<br>
it. To add new patches they should be replies to patch 1. But then<br>
there's the magic involved where patchwork tries to figure out whether<br>
it's an additional patch or a new version of patch 1...<br>
<br>
It's much simpler to just send the whole series as a whole. And please<br>
squash patches 2+3 together, from a quick read 3 seems to be just<br>
fixing patch 2.<br>
<br>
<br>
-- <br>
Petri Latvala<br>
<br>
<br>
> <br>
> Thank you,<br>
> Jake Freeland<br>
> <br>
> On Wed, Aug 17, 2022 at 9:23 AM Jake Freeland <<a href="mailto:jake@technologyfriends.net" target="_blank">jake@technologyfriends.net</a>><br>
> wrote:<br>
> <br>
> > ---<br>
> >  scripts/code_cov_gen_report | 9 +++------<br>
> >  tests/generate_testlist.sh  | 2 +-<br>
> >  2 files changed, 4 insertions(+), 7 deletions(-)<br>
> ><br>
> > diff --git a/scripts/code_cov_gen_report b/scripts/code_cov_gen_report<br>
> > index ffc8142e..b0696c51 100755<br>
> > --- a/scripts/code_cov_gen_report<br>
> > +++ b/scripts/code_cov_gen_report<br>
> > @@ -138,8 +138,7 @@ if [ "$MODE" != "info" ]; then<br>
> >                 echo "Creating per-file info files..."<br>
> >                 echo -n "" >${MERGED_INFO}.info<br>
> >                 for i in $RESULTS/*.tar.gz; do<br>
> > -                       TITLE=$(basename $i)<br>
> > -                       TITLE=$(basename $TITLE .tar.gz)<br>
> > +                       TITLE=$(basename "$i" .tar.gz)<br>
> ><br>
> >                         echo "Adding results from $i..."<br>
> >                         tar xf $i<br>
> > @@ -155,8 +154,7 @@ if [ "$MODE" != "info" ]; then<br>
> ><br>
> >                 TITLE=${MERGED_INFO}<br>
> >         else<br>
> > -               TITLE=$(basename $RESULTS)<br>
> > -               TITLE=$(basename $TITLE .tar.gz)<br>
> > +               TITLE=$(basename "$RESULTS" .tar.gz)<br>
> ><br>
> >                 echo "Adding results from $RESULTS..."<br>
> >                 tar xf $RESULTS<br>
> > @@ -177,8 +175,7 @@ else<br>
> >                 echo "Copying $RESULTS to $DEST_DIR..."<br>
> >                 cp $RESULTS .<br>
> ><br>
> > -               TITLE=$(basename $RESULTS)<br>
> > -               TITLE=$(basename $TITLE .info)<br>
> > +               TITLE=$(basename "$RESULTS" .info)<br>
> >         fi<br>
> >  fi<br>
> ><br>
> > diff --git a/tests/generate_testlist.sh b/tests/generate_testlist.sh<br>
> > index 240a7eb5..ea999a45 100755<br>
> > --- a/tests/generate_testlist.sh<br>
> > +++ b/tests/generate_testlist.sh<br>
> > @@ -15,4 +15,4 @@ while [ $# -gt 0 ] ; do<br>
> >         shift<br>
> >  done<br>
> ><br>
> > -printf "\nEND TESTLIST" >> $OUTPUT<br>
> > +printf "\nEND TESTLIST\n" >> $OUTPUT<br>
> > --<br>
> > 2.37.2<br>
> ><br>
> ><br>
</blockquote></div>