jazz_hands icon indicating copy to clipboard operation
jazz_hands copied to clipboard

Allow specify including/excluding debugger

Open snwfog opened this issue 11 years ago • 12 comments

Its conflicting with RubyMine's own debugger, but I still want to use the console with Pry for debugging simultaneously.

snwfog avatar Jul 18 '13 19:07 snwfog

This is a problem on our team as well. I think it is less of an issue with including debugger and more of an issue with Bundler and not being able to have a local gemfile that doesn't get checked in.

localredhead avatar Jul 18 '13 20:07 localredhead

Hrm... with all the incompatibility with 2.0 and others asking for byebug, I think I'll create a branch without pry-debugger and folks can drop in whichever debug mechanism they want.

@localredhead Can't you just set the BUNDLE_GEMFILE environment variable to Gemfile.local and gitignore it?

nixme avatar Jul 18 '13 20:07 nixme

@nixme that sounds far too easy to be reasonable! :) I'll suggest that to the RubyMine users in group.

A branch that is void of pry-debugger sounds like the best scenario - but for me I liked the fact that it just worked when it was pre-bundled.

localredhead avatar Jul 18 '13 21:07 localredhead

+1 I would love a branch without debugger for now. Until debugger support works in Ruby 2.0 it would be great to have an easier alternative with pry-byebug as mentioned in https://github.com/nixme/jazz_hands/issues/11

theoretick avatar Jul 27 '13 17:07 theoretick

@nixme Did you ever get any further with the branch that excludes debugger? I'm refreshing gems on a project, and would like to move away from some of the forks necessary to get jazz_hands working on 2.0 correctly.

wyaeld avatar Oct 21 '13 01:10 wyaeld

Okay. A branch based off 0.5.2 is available at https://github.com/nixme/jazz_hands/tree/bring-your-own-debugger

You can use it in bundler via:

gem 'jazz_hands', github: 'nixme/jazz_hands', branch: 'bring-your-own-debugger'

I might release that combined with @jhre's #13 as major release 1.0 if this works for everyone.

nixme avatar Oct 24 '13 18:10 nixme

I've been able to come back and revisit this, running the following apears to work fine, this is on ruby 2.0.0

gem 'jazz_hands', github: 'nixme/jazz_hands', branch: 'bring-your-own-debugger'
gem 'pry-byebug'

+1 for releasing this with @jhre's work.

wyaeld avatar Jan 16 '14 02:01 wyaeld

We've been running the bring-your-own branch for a good while now and it has worked flawlessly. It's especially handy when you want to run ruby-head as the debugger dependency pulls in debugger-ruby_core_source which is not (and never will be, which is proper of course) updated to ruby-head. Any chance of seeing this behavior merged to master?

dvdplm avatar Jan 29 '14 09:01 dvdplm

:+1: would love to see this merged on master.

GriffinHeart avatar Apr 21 '14 13:04 GriffinHeart

I across this issue today and sort of confused why it still occurs with a fresh attempt to use jazz_hands on a freshly set-up laptop; debugger 1.6.8 was something that forced me unable to finish a bundle, but it seems pry-debugger should have been removed from jazz_hands already?

lozandier avatar Jul 06 '14 05:07 lozandier

@lozandier Have you specify the correct branch in your Gemfile?

gem 'jazz_hands', github: 'nixme/jazz_hands', branch: 'bring-your-own-debugger'

snwfog avatar Jul 06 '14 13:07 snwfog

That's what I did after trying out gem 'jazz_hands and it worked, but I thought jazz_hands incorporated such changes into the master branch; this issue ticket was closed with a commit that suggested this had happened, but apparently not with the version of the gem that is installed by default with the use of `gem jazz producing the errors described with this ticket.

On Sun, Jul 6, 2014 at 6:09 AM, Charles Yang [email protected] wrote:

@lozandier https://github.com/lozandier Have you specify the correct branch in your Gemfile?

gem 'jazz_hands', github: 'nixme/jazz_hands', branch: 'bring-your-own-debugger'

— Reply to this email directly or view it on GitHub https://github.com/nixme/jazz_hands/issues/12#issuecomment-48111424.

Kevin Lozandier [email protected] [email protected]

lozandier avatar Jul 06 '14 20:07 lozandier