Hi

On Fri, Aug 5, 2022 at 2:39 PM Paolo Bonzini <pbonzini@redhat.com> wrote:

> On 7/12/22 11:35, marcandre.lureau@redhat.com wrote:
> > From: Marc-André Lureau <marcandre.lureau@redhat.com>
> >
> > A subproject test may be simply in the "PROJECT" suite (such as
> > "qemu-common" with the following patches)
> >
> > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> > ---
> >   scripts/mtest2make.py | 7 +++++--
> >   1 file changed, 5 insertions(+), 2 deletions(-)
> >
> > diff --git a/scripts/mtest2make.py b/scripts/mtest2make.py
> > index 0fe81efbbcec..606821ee2732 100644
> > --- a/scripts/mtest2make.py
> > +++ b/scripts/mtest2make.py
> > @@ -51,8 +51,11 @@ def process_tests(test, targets, suites):
> >
> >       test_suites = test['suite'] or ['default']
> >       for s in test_suites:
> > -        # The suite name in the introspection info is "PROJECT:SUITE"
> > -        s = s.split(':')[1]
> > +        # The suite name in the introspection info is "PROJECT" or
> "PROJECT:SUITE"
> > +        try:
> > +            s = s.split(':')[1]
> > +        except IndexError:
> > +            continue
>
> Shouldn't you continue with s begin simply "PROJECT"?  That is, just
>
>      if ':' in s:
>          s = s.split(':')[1]
>
> This way you can do "make check-qemu-common".
>
> >           if s == 'slow' or s == 'thorough':
> >               continue
> >           if s.endswith('-slow'):
>
> and then however these two "ifs" need to be under the case where the
> suite name is "PROJECT:SUITE".
>
>
Thanks for the tips, updated


-- 
Marc-André Lureau