| From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
|---|---|
| To: | Melih Mutlu <m(dot)melihmutlu(at)gmail(dot)com> |
| Cc: | pgsql-hackers(at)postgresql(dot)org, Andres Freund <andres(at)anarazel(dot)de>, 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-07-28 22:44:28 |
| Message-ID: | 20220728224428.GN15006@telsasoft.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
I think the "only_if" should allow separately running one but not both of the
windows instances, like:
+ only_if: $CIRRUS_CHANGE_MESSAGE !=~ '.*\nci-os-only:.*' || $CIRRUS_CHANGE_MESSAGE =~ '.*\nci-os-only:[^\n]*mingw64'
I'm not sure, but maybe this task should only run "by request", and omit the
first condition:
+ only_if: $CIRRUS_CHANGE_MESSAGE =~ '.*\nci-os-only:[^\n]*mingw64'
I think it should include something like
+ setup_additional_packages_script: |
+ REM C:\msys64\usr\bin\pacman.exe -S --noconfirm ...
Let's see what others think about those.
Do you know if this handles logging of crash dumps ?
With tweaks to prep_buildtree, and with ./configure --cache-file, that step
goes down to ~36sec (unless configure needs to be re-run).
I also looked into using busybox to avoid running separate processes for each
"ln", but I think 36sec is good enough.
At one point, I tried setting "CIRRUS_SHELL: bash" to avoid writing "bash -c"
over and over, but never got it working.
--
Justin
| Attachment | Content-Type | Size |
|---|---|---|
| 0001-Added-Windows-with-MinGW-environment-in-Cirrus-CI.patch | text/x-diff | 3.9 KB |
| 0002-f.patch | text/x-diff | 4.3 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nathan Bossart | 2022-07-28 22:46:18 | Re: [Patch] Fix bounds check in trim_array() |
| Previous Message | Thomas Munro | 2022-07-28 22:29:41 | Re: Cygwin cleanup |