[PATCH v7 0/4] Add ACVP Tool

Aaron Conole aconole at redhat.com
Thu Apr 13 23:20:13 CEST 2023


Jeremy Spewock <jspewock at iol.unh.edu> writes:

> Hello,
>
> I can see why you would want to squash everything into one patch; I had just thought that the intention with
> maintaining the structure of the old patch series was to preserve Brandon's original commits and comments alongside
> some minor expansion where necessary. If it would be preferred however, I can resubmit with everything squashed into
> a single patch.
>
> Aaron, if you could also review this series and give your thoughts on the matter as well that would be greatly
> appreciated.

After looking at the structure, I agree with Thomas about squashing.
Generally, try to introduce the change as a complete atom.  In this
case, we don't extend anything anywhere else.  If there were other
scripts, etc. then we could do that.

When I see the squashed patch come through, I can apply and push.

Thanks!

> Thanks,
> Jeremy
>
> On Wed, Apr 12, 2023 at 4:16 AM Thomas Monjalon <thomas at monjalon.net> wrote:
>
>  Hi,
>
>  Aaron is the maintainer.
>  I can also push the commits.
>
>  I think you should squash all in one patch
>  and provide an explanation in the commit message
>  about what this tool is for exactly.
>
>  11/04/2023 23:22, Patrick Robb:
>  > Hi Ali,
>  > 
>  > Aaron says he is not a maintainer either. Do you know whether there is some
>  > place I can look to see who the current maintainers are? There isn't an
>  > equivalent to the MAINTAINERS file like in the main DPDK repo.
>  > 
>  > Thomas, you might be the only current maintainer on this repo. We'll be
>  > grateful at the lab if you're able to merge this patch.
>  > 
>  > Best,
>  > Patrick
>  > 
>  > On Mon, Apr 10, 2023 at 11:19 AM Ali Alnubani <alialnu at nvidia.com> wrote:
>  > 
>  > > > Hi Ali,
>  > > >
>  > > > Is this good to be merged since it's been acked by a maintainer with no
>  > > more discussion afterwards? Or is there something more to be done here?
>  > > [..]
>  > >
>  > > Hello Patrick,
>  > >
>  > > Already acked from my side, waiting for a maintainer to merge if they have
>  > > no other comments (Thomas/Aaron).
>  > >
>  > > Thanks,
>  > > Ali



More information about the ci mailing list