Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP Fix gdb debugging and test it on Travis CI #52

Closed
wants to merge 1 commit into from
Closed

Conversation

jdemeyer
Copy link
Collaborator

@jdemeyer jdemeyer commented Mar 30, 2017

The cysignals-CSI is currently broken for different reasons:

@jdemeyer jdemeyer force-pushed the travis_gdb branch 9 times, most recently from 8d015e1 to d2e5ae8 Compare March 30, 2017 13:37
@jdemeyer jdemeyer changed the title Test gdb on Travis CI WIP Test gdb on Travis CI Apr 1, 2017
@jdemeyer jdemeyer changed the title WIP Test gdb on Travis CI WIP Fix gdb debugging and test it on Travis CI Apr 1, 2017
@anntzer
Copy link

anntzer commented Aug 2, 2017

Also, would it be possible to have a way to just disable gdb debugging? I mostly just want to use cysignals', well, signal handling facility, and only work on Py3, so at that point the tracebacks generated by gdb failures are just noise to me.

@jdemeyer
Copy link
Collaborator Author

jdemeyer commented Sep 5, 2017

Also, would it be possible to have a way to just disable gdb debugging?

What do you have in mind? A compile-time setting, run-time setting, environment variable...?

@anntzer
Copy link

anntzer commented Sep 5, 2017

I think gdb integration should be turned on either by an environment variable (my preference, as it is the easiest to set without modifying the source code) or by a runtime function (e.g. cysignals.run_gdb_on_failure()). Or possibly even by a C macro, e.g. NDEBUG (not sure I really like that idea).

@kliem kliem deleted the branch master December 10, 2021 20:00
@kliem kliem closed this Dec 10, 2021
@collares
Copy link

testgdb.py says it "is known to be broken in some cases, see [this issue]", but this issue was closed. Does this mean previously-known issues were fixed, or just are no longer relevant?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants