diff options
| author | Andres Freund <andres@anarazel.de> | 2022-09-21 17:15:54 -0700 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2022-09-21 17:16:51 -0700 |
| commit | 0c400445dbe316d67d19f79fdd096f0b77458b86 (patch) | |
| tree | ef4bfb6dfc6da3687632515a07f528eea8839a93 | |
| parent | 08ddb3c0852d87394b44eea45709b9d9b8154f68 (diff) | |
ci: windows: set error mode to not include SEM_NOGPFAULTERRORBOX
Cirrus defaults to SetErrorMode(SEM_NOGPFAULTERRORBOX | ...). That prevents
crash reporting from working unless binaries do SetErrorMode()
themselves. Furthermore, it appears that either python or, more likely, the C
runtime has a bug where SEM_NOGPFAULTERRORBOX can very occasionally *trigger*
a crash on process exit - which is hard to debug, given that it explicitly
prevents crash dumps from working...
Discussion: https://postgr.es/m/20220909235836.lz3igxtkcjb5w7zb%40awork3.anarazel.de
Backpatch: 15-, where CI was added
| -rw-r--r-- | .cirrus.yml | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/.cirrus.yml b/.cirrus.yml index df91b893537..fe8da021eda 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -386,6 +386,15 @@ task: # currently have a tool for that... CIRRUS_ESCAPING_PROCESSES: 1 + # Cirrus defaults to SetErrorMode(SEM_NOGPFAULTERRORBOX | ...). That + # prevents crash reporting from working unless binaries do SetErrorMode() + # themselves. Furthermore, it appears that either python or, more likely, + # the C runtime has a bug where SEM_NOGPFAULTERRORBOX can very + # occasionally *trigger* a crash on process exit - which is hard to debug, + # given that it explicitly prevents crash dumps from working... + # 0x8001 is SEM_FAILCRITICALERRORS | SEM_NOOPENFILEERRORBOX + CIRRUS_WINDOWS_ERROR_MODE: 0x8001 + only_if: $CIRRUS_CHANGE_MESSAGE !=~ '.*\nci-os-only:.*' || $CIRRUS_CHANGE_MESSAGE =~ '.*\nci-os-only:[^\n]*windows.*' windows_container: |