Skip to content
This repository

Sadness #9

Closed
trans opened this Issue March 11, 2012 · 5 comments

5 participants

7rans Gopal Patel Jordon Bedwell Alex Southgate John Mair
7rans
trans commented March 11, 2012
trans@logisys:confection$ qed -t 20 qed/verify.rdoc 
Started
..
From: /home/trans/Desktop/Programs/rubyworks/confection/qed/verify.rdoc @ line 17:

    12:     config = confection(:foo)
    13: 
    14:     assert Confection::Controller === config
    15: 
    16:     require 'pry'
 => 17:     binding.pry
    18: 
    19:     # TODO: Bug in AE? This doesn't work, and calls Controller#text somhow.
    20:     Confection::Controller.assert === config
    21: 
    22: Then if we execute it using `#call` it will be evaluate in the current

[1] pry(#<QED::Scope>)> step
ArgumentError: uncaught throw :breakout_nav
from /home/trans/.rbfu/rubies/1.9.3-p0/lib/ruby/gems/gems/pry-nav-0.1.0/lib/pry-nav/commands.rb:27:in `throw'
Jordon Bedwell

I second this, happens on every one of my apps (on 1.9.3 for me). I can't use Pry-Nav because it doesn't work for me.

Alex Southgate

Ditto the above, as well as occasional segfaults when next-ing or step-ing

John Mair
Collaborator

@fromagie the current solution is to require 'pry-nav' explicitly in the code prior to a binding.pry

As for the segfaults this is certainly a bug in Ruby itself -- pry-nav is pure ruby and should never cause a segfault no matter what happens.

Gopal Patel
Owner
nixme commented March 15, 2012

@banister, Is this some sort of loading order issue with my Pry.start monkeypatch?

John Mair
Collaborator

@nixme, Yes, basically plugins are loaded too late for your Pry.start monkeypatch to take effect. I'll try to move plugin loading to an earlier stage for the next release, see the discussion here: pry/pry#489

Gopal Patel nixme closed this issue from a commit April 13, 2012
John Mair added require 'pry-nav' to cli.rb, fixes #9
* Pry loads the cli.rb for a plugin at startup, rather than at Pry.start().
By adding this file we ensure that pry-nav's monkeypatch applies without requiring an additional
require 'pry-nav' by the user.
8d679f8
Gopal Patel nixme closed this in 8d679f8 April 15, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.