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

ipython5 use TerminalInteractiveShell debugger_cls solution for #105 #108

Merged
merged 2 commits into from
Jan 25, 2017

Conversation

JBKahn
Copy link
Contributor

@JBKahn JBKahn commented Aug 6, 2016

A solution for #105
It seems like a good place to start getting the ball rolling. Thoughts?

from IPython.core.debugger import BdbQuit_excepthook
from IPython.terminal.interactiveshell import TerminalInteractiveShell
debugger_cls = TerminalInteractiveShell().debugger_cls
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alternatively I was going to define it as a function rather than at import time, which now that I think about it, sounds much better.

@JBKahn
Copy link
Contributor Author

JBKahn commented Sep 5, 2016

@gotcha just a friendly reminder.

@jakul
Copy link

jakul commented Oct 18, 2016

I have this problem too, what can we do to get this fix merged?

@bar
Copy link

bar commented Dec 1, 2016

@JBKahn @gotcha any reason for not merging the fix? danke

@JBKahn
Copy link
Contributor Author

JBKahn commented Dec 1, 2016

@bar I think it's just a matter of @gotcha currently being inactive. None of the other PRs or issues are currently progressing either. Hopefully he'll return soon.

@sinanm89
Copy link

This fixes ipdb completely for me when im trying to work with twisted code. Thanks!

@gnebehay
Copy link
Collaborator

Hi, I'm a new collaborator on ipdb. I'm looking to clean up all old issues and PRs. This one seems rather important. Could you give me just a brief executive summary why using debugger_cls solves the problem, I somehow fail to see the point.

@gnebehay
Copy link
Collaborator

Ah, I see, there is this line in https://github.com/ipython/ipython/blob/master/IPython/terminal/interactiveshell.py#L115

    @property
    def debugger_cls(self):
        return Pdb if self.simple_prompt else TerminalPdb

Ok, merge is in the making.

@gnebehay gnebehay closed this Jan 20, 2017
@gnebehay gnebehay reopened this Jan 20, 2017
@gnebehay
Copy link
Collaborator

gnebehay commented Jan 20, 2017

I merged the changes here: https://github.com/gotcha/ipdb/tree/fix-use-debugger_cls Could you check if that works for you?

@JBKahn
Copy link
Contributor Author

JBKahn commented Jan 20, 2017

Will check this weekend, thanks!

@JBKahn
Copy link
Contributor Author

JBKahn commented Jan 24, 2017

@gnebehay seems to work just fine with a quick test 👍

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

Successfully merging this pull request may close these issues.

5 participants