On Mon, Jan 21, 2019 at 11:53 PM Philippe Mathieu-Daudé
<phi...@redhat.com> wrote:
>
> On 1/21/19 6:15 PM, Alex Bennée wrote:
> > The "make check" target calls check-qtest which has the appropriate
> > system binaries as dependencies so we shouldn't need to do two steps
> > of make invocation.

Now I remember, this was necessary to do this in 2 steps before
ebb61f804d6, build threaded and test not threaded.
Maybe worth adding a comment referencing this commit?

> >
> > Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
> > ---
> >  tests/vm/freebsd | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/tests/vm/freebsd b/tests/vm/freebsd
> > index 19a3729172..a85c866c30 100755
> > --- a/tests/vm/freebsd
> > +++ b/tests/vm/freebsd
> > @@ -25,7 +25,6 @@ class FreeBSDVM(basevm.BaseVM):
> >          cd $(mktemp -d /var/tmp/qemu-test.XXXXXX);
> >          tar -xf /dev/vtbd1;
> >          ./configure {configure_opts};
> > -        gmake --output-sync -j{jobs} {verbose};
> >          gmake --output-sync -j{jobs} check {verbose};
> >      """
> >
>
> Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>
>

Reply via email to