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

Edit rackup start command on readme.md to rackup -s Puma. #243

Merged
merged 1 commit into from Apr 28, 2013
Merged

Edit rackup start command on readme.md to rackup -s Puma. #243

merged 1 commit into from Apr 28, 2013

Conversation

hendrauzia
Copy link
Contributor

Steps to reproduce:
Create file config.ru in app root with the following content. (http://guides.rubyonrails.org/v2.3.11/rails_on_rack.html#rackup)

require "config/environment"
use Rails::Rack::LogTailer
use Rails::Rack::Static
run ActionController::Dispatcher.new

Run rackup using puma the web server.

rackup -s puma

Expected Result:
Application runs with puma as it's web server.

Actual Result:

.bundle/gems/rack-1.1.6/lib/rack/handler.rb:21:in `const_get': wrong constant name puma (NameError)

Solution:
Run with rackup -s Puma, NOT rackup -s puma

Steps to reproduce:
1. Create config.ru. (http://guides.rubyonrails.org/v2.3.11/rails_on_rack.html#rackup)
2. rackup -s puma

Expected Result:
Application runs with puma as it's web server.

Actual Result:
.bundle/gems/rack-1.1.6/lib/rack/handler.rb:21:in `const_get': wrong constant name puma (NameError)

Solution:
Start with rackup -s Puma, NOT rackup -s puma.
evanphx added a commit that referenced this pull request Apr 28, 2013
Edit rackup start command on readme.md to rackup -s Puma.
@evanphx evanphx merged commit 7856eab into puma:master Apr 28, 2013
@hendrauzia hendrauzia deleted the patch-1 branch April 29, 2013 02:50
MSP-Greg added a commit to MSP-Greg/puma that referenced this pull request Jul 30, 2020
MSP-Greg added a commit to MSP-Greg/puma that referenced this pull request Jul 30, 2020
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

2 participants