Re: ci: Macos failures due to MacPorts behaviour change
Nazir Bilal Yavuz <byavuz81@gmail.com>
From: Nazir Bilal Yavuz <byavuz81@gmail.com>
To: Andres Freund <andres@anarazel.de>
Cc: Thomas Munro <thomas.munro@gmail.com>, pgsql-hackers@postgresql.org
Date: 2024-11-27T13:33:02Z
Lists: pgsql-hackers
Attachments
- v3-0001-ci-PG-17-16-Fix-cached-MacPorts-installation-mana.patch (text/x-patch) patch v3-0001
- v3-0001-ci-PG-15-Fix-cached-MacPorts-installation-managem.patch (text/x-patch) patch v3-0001
Hi, On Fri, 22 Nov 2024 at 20:28, Andres Freund <andres@anarazel.de> wrote: > The right approach probably is to include the list of packages in the key. A > bit annoying to change, because we'd need to move the list of packages to an > environment variable or file, but doable. I think something like > > env: > MACOS_PACKAGE_LIST: >- > ccache > icu > ... > fingerprint_script: | > ... > echo $MACOS_PACKAGE_LIST > ... > setup_additional_packages_script: | > sh src/tools/ci/ci_macports_packages.sh $MACOS_PACKAGE_LIST > > should work? I think this is a nice solution and it worked successfully [1]. Now, REL_[17 | 16]_* and master branches use the same cache which is different from the REL_15_STABLE branch's cache. In case you want to continue with this, the patches are attached. I merged 'using a loop in the install script' from Thomas' patch and the change above. [1] First run - second run (See cache is used in the second run.) [Master] https://cirrus-ci.com/task/6398434171682816 - https://cirrus-ci.com/task/6460963865493504 [PG 16] https://cirrus-ci.com/task/5697896752873472 - https://cirrus-ci.com/task/4656279002546176 [PG 15] https://cirrus-ci.com/task/5192066743926784 - https://cirrus-ci.com/task/5033544097988608 -- Regards, Nazir Bilal Yavuz Microsoft
Commits
-
ci: Fix cached MacPorts installation management
- 262283d5eec3 18.0 landed
- 3bbfa2ef785a 15.11 landed
- c26831e73624 16.7 landed
- c93dffd89ba5 17.3 landed