pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
pgsql: ci: windows: set error mode to not include SEM_NOGPFAULTERRORBOX
2+ messages / 1 participants
[nested] [flat]

* pgsql: ci: windows: set error mode to not include SEM_NOGPFAULTERRORBOX
@ 2022-09-22 00:18 Andres Freund <andres@anarazel.de>
  0 siblings, 0 replies; 2+ messages in thread

From: Andres Freund @ 2022-09-22 00:18 UTC (permalink / raw)
  To: pgsql-committers@lists.postgresql.org

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

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/661ee7bfc661a4ba52ed57cff46e414b2add75b1

Modified Files
--------------
.cirrus.yml | 9 +++++++++
1 file changed, 9 insertions(+)



^ permalink  raw  reply  [nested|flat] 2+ messages in thread

* pgsql: ci: windows: set error mode to not include SEM_NOGPFAULTERRORBOX
@ 2022-09-22 00:18 Andres Freund <andres@anarazel.de>
  0 siblings, 0 replies; 2+ messages in thread

From: Andres Freund @ 2022-09-22 00:18 UTC (permalink / raw)
  To: pgsql-committers@lists.postgresql.org

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

Branch
------
REL_15_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/0c400445dbe316d67d19f79fdd096f0b77458b86

Modified Files
--------------
.cirrus.yml | 9 +++++++++
1 file changed, 9 insertions(+)



^ permalink  raw  reply  [nested|flat] 2+ messages in thread


end of thread, other threads:[~2022-09-22 00:18 UTC | newest]

Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2022-09-22 00:18 pgsql: ci: windows: set error mode to not include SEM_NOGPFAULTERRORBOX Andres Freund <andres@anarazel.de>
2022-09-22 00:18 pgsql: ci: windows: set error mode to not include SEM_NOGPFAULTERRORBOX Andres Freund <andres@anarazel.de>

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox