Opened 2 years ago
Last modified 2 years ago
#65630 closed enhancement
GraphicsMagick: disable SIGSEGV handler in library; add debug variant — at Version 4
Reported by: | mascguy (Christopher Nielsen) | Owned by: | ryandesign (Ryan Carsten Schmidt) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.7.2 |
Keywords: | debug | Cc: | |
Port: | GraphicsMagick |
Description (last modified by mascguy (Christopher Nielsen))
Presently, we're trying to diagnose a crash occurring for darktable
, which appears to be originating from within this port:
There are two barriers, though:
- The GraphicsMagick library installs a SIGSEGV handler, preventing the app from catching/handling the condition;
- No
+debug
variant, which would assist with diagnostics.
Relative to the signal handler, we should consider patching the code to disable that, for the library. Along with engaging with upstream, for a longer-term fix.
As for the debug variant, that should be reasonably straightforward, and would be beneficial.
Change History (4)
comment:1 Changed 2 years ago by mascguy (Christopher Nielsen)
Cc: | ryandesign removed |
---|---|
Owner: | changed from mascguy to ryandesign |
comment:2 Changed 2 years ago by mascguy (Christopher Nielsen)
Resolution: | → wontfix |
---|---|
Status: | assigned → closed |
issue:64252 has been resolved, so this is no longer needed.
If there's user demand, we can always reopen.
comment:3 Changed 2 years ago by jmroot (Joshua Root)
There is still the valid issue raised in the other ticket that GraphicsMagick installs its own SIGSEGV handler when used as a library. Up to you if you want to open a ticket for that, here or upstream.
comment:4 Changed 2 years ago by mascguy (Christopher Nielsen)
Description: | modified (diff) |
---|---|
Resolution: | wontfix |
Status: | closed → reopened |
Summary: | GraphicsMagick: add debug variant → GraphicsMagick: disable SIGSEGV handler in library; add debug variant |
Let's expand the scope of this ticket to include the lib signal handler, per @jmroot in comment:3
Too much on my plate, I'll let Ryan tackle this