Re: Mingw task for Cirrus CI

From: Melih Mutlu <m(dot)melihmutlu(at)gmail(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>, Justin Pryzby <pryzby(at)telsasoft(dot)com>, pgsql-hackers(at)postgresql(dot)org, Andrew Dunstan <andrew(at)dunslane(dot)net>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
Subject: Re: Mingw task for Cirrus CI
Date: 2022-10-19 14:59:22
Message-ID: CAGPVpCRLj23LCftUsJVQ1equ0FTbhu8sjL7sckVXS6Wcy5OA1Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

Andres Freund <andres(at)anarazel(dot)de>, 19 Eki 2022 Çar, 06:19 tarihinde şunu
yazdı:

> It's a bit odd to separate the CCACHE_* variables from each other
> (e.g. BUILD_DIR is inbetween them)...
>
> Perhaps add a comment explaining that otherwise plpython tests fail?

>
With this src/tools/ci/cores_backtrace.sh shouldn't need to be modified
> anymore afaict?
>
> I'd replace --buildtype debug with -Ddebug=true -Doptimization=g.
>

Seems like this could use %MTEST_ARGS%?

All the things you mentioned above are done.

> > + CCACHE_SLOPPINESS: pch_defines,time_macros
> > + CCACHE_DEPEND: 1
>
> I experimented a bit and it looks like ccache doesn't yet quite work in CI,
> but only because the ccache needs to be larger. Looks like we need about
> ~400MB.
>
> A fully cached build is ~2min
>

Then I should increase CCACHE_MAXSIZE, right? Made it 500MB for MinGW.

Sharing the updated version of the patch.

Thanks,
Melih

Attachment Content-Type Size
v9-0001-Added-Windows-with-MinGW-environment-in-Cirrus-CI.patch application/octet-stream 4.9 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Önder Kalacı 2022-10-19 15:18:14 Re: [Proposal] Add foreign-server health checks infrastructure
Previous Message Robert Haas 2022-10-19 14:48:03 Re: thinko in basic_archive.c