search

gma / nesta

File Based CMS and Static Site Generator
http://nestacms.com
MIT License
902 stars 121 forks source link

Remove shotgun as a dependency #125

Closed datenimperator closed 10 years ago

datenimperator commented 11 years ago

The shotgun gem shouldn't be a dependency. For example, sinatra-contrib contains sinatra/reloader which is much faster than shotgun and does not fork (therefore: works on Windows) Choosing a reloader isn't nestas responsibility.

gma commented 11 years ago

I agree. If you do a pull request, could you make sure that shotgun is a dependency of generated projects (i.e. the site has shotgun in its Gemfile) instead? That's important, or bundle exec shotgun won't work for new sites.

We're actually going to move away from shotgun in the docs to @micahchalmer's mr-sparkle (which is loads faster, and generally awesome), so if anybody does pick this up before I get to it, that'd be a good way to go. Reasoning for using mr-sparkle instead of sinatra/reloader is covered in this mailing list thread:

http://librelist.com/browser//nesta/2012/12/6/external-access-to-staging-site/#2c220f07f5a8facac7bed3070454b0c9

mr-sparkle was written as a result, and was announced in a subsequent thread.