search

documentcloud / jammit

Industrial Strength Asset Packaging for Rails
http://documentcloud.github.io/jammit/
MIT License
1.16k stars 197 forks source link

:( "Java is required!? DUH!" #223

Open iamvery opened 12 years ago

iamvery commented 12 years ago

Please update your docs / website to more expressly indicate that Java is a hard requirement for JS compression. Its "duh" now that I think about it, but I spent several hours trying to figure out why compression wasn't working on a server, but worked great everywhere else.

I understand that in general people "just know" that Java is a requirement for YUI, etc, but if it slips your mind or (in my case) you didn't know that, it can be very confusing that compression "gracefully" fails without any error or warning. We should probably at least see a warning in the log that compression was skipped due to Java missing.

Thanks for your great library!

(please try to overlook my extended emotions in this point.. its been one of those afternoons :)

rchampourlier commented 12 years ago

Well, I want to thank @iamvery for this issue, since I faced the same problem... The java dependency is far from evident to me since I discovered web-development in a ruby-only environment. A notice in logs or doc would be nice indeed...

[EDIT] Well in fact it gets logged... in the application-server's log, not in the Rails' one (in my case log/thin.0.log, not in log/rails.log).