Re: CI, macports, darwin version problems

From: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
To: Joe Conway <mail(at)joeconway(dot)com>
Cc: Andres Freund <andres(at)anarazel(dot)de>, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Christophe Pettus <christophe(dot)pettus(at)pgexperts(dot)com>
Subject: Re: CI, macports, darwin version problems
Date: 2024-07-25 02:53:53
Message-ID: CA+hUKGLAT-Ok9_=2iC3+h+GZ+HAB-XA8kHStzEwDA88gsn5VvQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Jul 25, 2024 at 11:35 AM Thomas Munro <thomas(dot)munro(at)gmail(dot)com> wrote:
> Looking good! Thanks. I have now pushed the patch to switch CI to
> Sonoma, back-patched as far as 15. Let's see how that goes. I have
> also paused the pgx machine for now, until Christophe is available to
> help us fix it.

Cfbot builds are working nicely on Sonoma.

But... unfortunately the github.com/postgres/postgres CI was working
for REL_15_STABLE only, and not 16, 17 or master. After scratching my
head for a moment, I realised that the new logic for finding the most
recent version of MacPorts is now picking up a new beta version of
MacPorts that has just been published, and apparently it doesn't work
quite right and couldn't install meson. D'oh!

I have pushed a fix for that: I went back to requesting 2.9.3 until we
are ready to change it.

Long version: I had imagined we might one day need to nail down the
version in a commented out line already, I just failed to think about
non-release versions appearing in the list. An alternative might be
to use a pattern that matches stable releases eg [0-9][0-9.]* to
exclude stuff like 2.10-beta1 but for now I have cold feet about that
lack of control. While thinking about that, it occurred to me that it
might also be better if it also re-installs from scratch whenever our
script that installs MacPorts changes, so I included its md5 in the
cache key. Otherwise it's a bit hard to test, since the cached
installation survives across rebuilds by design (that's why cfbot
isn't affected, it installed 2.9.3 before they unleashed 2.10-beta1
and cached the result). This way, if someone changes 2.9.3 to
2.whatever in that script, it'll do a fresh installation of MacPorts
on the first build in a given github account, and then later builds
will work from the cached copy. Seems like desirable behaviour for
future maintenance work.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Nathan Bossart 2024-07-25 02:58:52 Re: optimizing pg_upgrade's once-in-each-database steps
Previous Message Michael Paquier 2024-07-25 02:19:08 Re: [PATCH] Add additional extended protocol commands to psql: \parse and \bind