Re: [PATCH 6/7] .gitlab-ci.d/windows.yml: Unify the prerequisite packages

[Thomas Huth]

On 29/10/2022 15.06, Bin Meng wrote:
> Hi Thomas,
>
> On Sat, Sep 24, 2022 at 5:20 PM Bin Meng <bmeng.cn@gmail.com> wrote:
> > Hi Thomas,
> >
> > On Sat, Sep 10, 2022 at 8:32 AM Bin Meng <bmeng.cn@gmail.com> wrote:
> > > On Sat, Sep 10, 2022 at 12:32 AM Thomas Huth <thuth@redhat.com> wrote:
> > > > On 08/09/2022 15.28, Bin Meng wrote:
> > > > > From: Bin Meng <bin.meng@windriver.com>
> > > > >
> > > > > At present the prerequisite packages for 64-bit and 32-bit builds
> > > > > are slightly different. Let's use the same packages for both.
> > > >
> > > > Not sure whether that's a good idea ... I did that on purpose to save some
> > > > few time during compilation (since the Windows jobs are running very long
> > > > already) ... did you check whether it makes a difference in the run time now?
> > >
> > > Not much difference on the build time. Actually I found after we
> > > switched to single thread build the time did not increase too.
> > >
> > > One side note regarding the gitlab shared runner:
> > >
> > > It seems the shared runner Windows VM is quite slow. Is it possible to
> > > get a faster VM externally?
> >
> > Any further comment for this patch?
>
> Ping?

As long as the Windows CI runners are still that slow, I'm still not 
convinced that it is a good idea to add more stuff here. Maybe to the 32-bit 
build, since it runs a bit faster, but I think we really should avoid to 
extend the 64-bit build.

 Thomas