Hi Marc-André,
On Wed, Nov 23, 2022 at 10:14 PM Marc-André Lureau
<marcandre.lur...@gmail.com> wrote:
>
> Hi Bin
>
> On Fri, Oct 28, 2022 at 9:06 AM Bin Meng <bin.m...@windriver.com> wrote:
> >
> > Now that we have fixed various test case issues as seen when running
> > on Windows, let's enable the qtest build on Windows.
> >
> > Signed-off-by: Bin Meng <bin.m...@windriver.com>
> > Reviewed-by: Thomas Huth <th...@redhat.com>
>
> We haven't solved the CI timing out or eating all the CPU time, right?
Correct
>
> Can we simply exclude it from CI for now, ie add to this patch
Thanks for your suggestion!
>
> diff --git a/.gitlab-ci.d/windows.yml b/.gitlab-ci.d/windows.yml
> index 093276ddbc..ba9045ec38 100644
> --- a/.gitlab-ci.d/windows.yml
> +++ b/.gitlab-ci.d/windows.yml
> @@ -62,7 +62,7 @@ msys2-64bit:
> - .\msys64\usr\bin\bash -lc './configure --target-list=x86_64-softmmu
> --enable-capstone'
> - .\msys64\usr\bin\bash -lc 'make'
> - - .\msys64\usr\bin\bash -lc 'make check || { cat
> build/meson-logs/testlog.txt; exit 1; } ;'
> + - .\msys64\usr\bin\bash -lc 'make check MTESTARGS="--no-suite
> qtest" || { cat build/meson-logs/testlog.txt; exit 1; } ;'
The double quote should be escaped.
>
> msys2-32bit:
> extends: .shared_msys2_builder
> @@ -96,4 +96,4 @@ msys2-32bit:
> - cd output
> - ..\msys64\usr\bin\bash -lc "../configure --target-list=ppc64-softmmu"
> - ..\msys64\usr\bin\bash -lc 'make'
> - - ..\msys64\usr\bin\bash -lc 'make check || { cat
> meson-logs/testlog.txt; exit 1; } ;'
> + - ..\msys64\usr\bin\bash -lc 'make check MTESTARGS="--no-suite
> qtest" || { cat meson-logs/testlog.txt; exit 1; } ;'
I don't think we need to touch 32-bit.
> Could you resubmit your missing win test patches and check if gitlab is happy?
>
Sure, will do soon.
Regards,
Bin
