Skip to content
This repository has been archived by the owner on Jul 23, 2023. It is now read-only.

Rack downgrade, thin upgrade, to co-exist with Rails 3.2 #109

Closed
wants to merge 1 commit into from
Closed

Rack downgrade, thin upgrade, to co-exist with Rails 3.2 #109

wants to merge 1 commit into from

Conversation

shimms
Copy link

@shimms shimms commented Apr 30, 2013

The default Rails 3.2 with thin installed rack ~> 1.4 and thin ~> 1.5.

Updated gemspec dependencies to happily co-exist with these gems, enabling co-habitation with a Rails gemset.

@stevegraham
Copy link
Owner

please see readme

@shimms
Copy link
Author

shimms commented Apr 30, 2013

Not sure what the readme is supposed to tell me here - I get that it isn't supposed to run "inside" a rails app, but if you've got a gemset for a rails app, you can't install Slanger with the current dependencies.

@stevegraham
Copy link
Owner

yep, it's not intended to be inside another application's gemset. it's packaged as a ruby gem only because that is a convenient method of distribution.

@shimms
Copy link
Author

shimms commented Apr 30, 2013

Cool - just saw the commentary in #81, explains the position clearer.

Can I suggest that the readme be updated to reflect this stance? I feel that the statement about being designed not to run inside Rails isn't as clear as it could be - took this to mean it isn't designed to be mounted/run as an initializer/etc ala: Sidekiq, Resque Web etc.

Appreciate your desire to keep it locked down and stand alone, but perhaps a bit more specificity in the readme would help reduce the PRs about gem versioning?

@stevegraham
Copy link
Owner

Good idea, it appears it isn't as clear as it needs to be because this class of PR seems to be one of the most frequent. Sorry for not being clearer.

@markburns
Copy link
Contributor

I'm just working on the README now

@markburns
Copy link
Contributor

Wait, sorry, commented on the wrong PR in a different tab.

Just to explain:

whilst my comment is relevant to this discussion, I thought this was going on right now for a second as I go through and review all the old PRs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants