Re: Mingw task for Cirrus CI

Поиск
Список
Период
Сортировка
От Justin Pryzby
Тема Re: Mingw task for Cirrus CI
Дата
Msg-id 20220728224428.GN15006@telsasoft.com
обсуждение исходный текст
Ответ на Re: Mingw task for Cirrus CI  (Melih Mutlu <m.melihmutlu@gmail.com>)
Ответы Re: Mingw task for Cirrus CI  (Thomas Munro <thomas.munro@gmail.com>)
Re: Mingw task for Cirrus CI  (Justin Pryzby <pryzby@telsasoft.com>)
Список 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

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Thomas Munro
Дата:
Сообщение: Re: Cygwin cleanup
Следующее
От: Nathan Bossart
Дата:
Сообщение: Re: [Patch] Fix bounds check in trim_array()