Re: [PATCH] Add Windows support for backtrace_functions (MSVC only) - Mailing list pgsql-hackers

From Álvaro Herrera
Subject Re: [PATCH] Add Windows support for backtrace_functions (MSVC only)
Date
Msg-id 202510300932.isdvgkzqzshk@alvherre.pgsql
Whole thread Raw
In response to Re: [PATCH] Add Windows support for backtrace_functions (MSVC only)  (Jakub Wartak <jakub.wartak@enterprisedb.com>)
Responses Re: [PATCH] Add Windows support for backtrace_functions (MSVC only)
List pgsql-hackers
On 2025-Oct-30, Jakub Wartak wrote:

> Hi Bryan, cfbot is red. I'm was fan of having those tests for this
> (bring complexity and we didn't have tests for Linux backtrace
> anyway), but now MINGW win32 is failing on those tests where the
> feature is not present:

I hate to say this after the code is written, but I think we should not
put any tests in the first step.  I predict that these are going to be
enormously brittle and that we'll waste a lot of time making them
stable.  I think we should commit the Windows support for backtraces
first, then consider whether we actually want TAP tests for the overall
feature.  We've gone several years with glibc backtrace support without
any tests -- why do we think the Windows implementation thereof _must_
necessarily have them?

-- 
Álvaro Herrera         PostgreSQL Developer  —  https://www.EnterpriseDB.com/
<Schwern> It does it in a really, really complicated way
<crab> why does it need to be complicated?
<Schwern> Because it's MakeMaker.



pgsql-hackers by date:

Previous
From: Bertrand Drouvot
Date:
Subject: Re: Skip unregistered custom kinds on stats load
Next
From: Bryan Green
Date:
Subject: Re: [PATCH] Add Windows support for backtrace_functions (MSVC only)