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

Please split out listen support to a separate gem #2347

Closed
henderea opened this issue Jul 13, 2017 · 6 comments
Closed

Please split out listen support to a separate gem #2347

henderea opened this issue Jul 13, 2017 · 6 comments

Comments

@henderea
Copy link

henderea commented Jul 13, 2017

When I updated my sass installation today, I found that it was trying to install a gem called sass-listen, which conflicted with the listen gem I already had installed. I only use the sass gem with rails, so I have no need for listen support in it.

Is there anyway we could make listen support a separate gem? I don't feel that I should have to have sass-listen installed just so I can use the latest version of sass with rails.

Also, if the listen support is split out, that means less bloat for people like me who don't need listen support for sass.

@cwonrails
Copy link

I noticed the same today when updating my global gems. Screenshot:
image

@nex3
Copy link
Contributor

nex3 commented Jul 13, 2017

Duplicate of #2345. There's no way to keep the existing CLI without depending on some version of listen, but I plan to release a version of sass-listen that doesn't conflict with the real listen.

@nex3 nex3 closed this as completed Jul 13, 2017
@fmluizao
Copy link

Sorry to comment on a closed issue, but I think this one is not a duplicate of #2345

This issue is about leave the decision to use or not file watching up to the user.

I think that sass-listen should depend on sass, not the inverse.

This way, people who need/want file watching will use sass-listen.

@bquorning
Copy link

Also, if the listen support is split out, that means less bloat for people like me who don't need listen support for sass.

I agree. I have no need for the “watch” support, and I would rather not add 3 extra gems (sass-listen, rb-fsevent and rb-inotify) to my application.

@joevandyk
Copy link

rb-ionotify also brings in the ffi gem.

@joevandyk
Copy link

@nex3 Any chance sass-listen could depend on sass? It can be a pain to install the dependencies for sass-listen, fb-fsevent, rb-ionotify, and ffi.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
6 participants