-
-
Notifications
You must be signed in to change notification settings - Fork 753
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
Editing config.rb while server is running and then exiting crashes Middleman #1430
Comments
Thanks for reporting |
I'm running a different but maybe related issue, when saving
and goes on with the undefined method error continuously |
@alex88 I’m seeing that also for example when changing livereload settings while the server is running. See middleman/middleman-livereload#71 (comment). |
@kasper I got that issue even without changing them, just by saving the |
@alex88 Yes, that also is enough. |
I've just tried without the livereload activated and the issue is the same update: tried also without having the gem in Gemfile, same behaviour |
Yeah, I think something may have changed slightly with Webrick... will investigate. |
This fixes the bug referenced. It may be somewhat broken, and perhaps this should be in unmount_instance. Basically what is happening is that the ensure block in Webrick.start runs, setting @shutdown_pipe to nil, then it tries to remount with a broken @webrick. Im not sure shy the ensure block is running exactly though...
This PR fixes the problem by stopping the webrick instance & recreating one on reload. Probably wrong, but it may give someone with more of a clue why this is happening. The PR has more info why this bug is happening. |
@tdreyno when will you release the version with this fix? |
There was no change. As per: #1475 (comment) this was a bug in Ruby 2.2.0 which is fixed here (ruby/ruby@ab0a64e) I guess we're waiting for Ruby 2.2.2 |
Oh great, I misread the comment and I thought you merged the fix |
I'm using |
@ashlynbaum Can you point your Gemfile at Github |
@tdreyno That did fix the bug, thanks. |
@tdreyno works for me. |
Just an FYI for people coming in via Google: |
I commented out the following and the error went away. Not sure why, anyone care enough to explain? #require 'rubygems' |
I'm doing some issue-gardening 🌿🌷🌾 and came across this issue. This should be fixed and the issue can be closed. |
Editing config.rb while server is running and then exiting crashes Middleman 3.3.7 on Yosemite 10.10.1 and Ruby 2.2.0.
The text was updated successfully, but these errors were encountered: