Skip to content

Fix missing dependency declarations (accused by sprockets_better_errors gem). #60

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

Merged
merged 5 commits into from
Dec 15, 2013

Conversation

luisalima
Copy link
Collaborator

I upgraded my sprockets to be able to install the sprockets_better_errors gem, which is great for detecting when we are doing something wrong in the rails asset pipeline.

This gem is missing dependency declarations for the pngs referenced in the app/assets/stylesheets/jquery.ui.theme.css.erb file.

This pull request adds the necessary dependency declarations.

For future reference

  • I'm running sprockets 2.2.2.backport2, sprockets-rails 2.0.0.backport1 and rails 3.2.16.
  • The error message for each file is:
Asset depends on 'jquery-ui/ui-bg_flat_75_ffffff_40x100.png' to generate properly but has not declared the dependency
Please add: `//= depend_on_asset "jquery-ui/ui-bg_flat_75_ffffff_40x100.png"` to '/Users/lu/.rvm/gems/ruby-2.0.0-p353@shiftit_rails3.2.16/gems/jquery-ui-rails-4.1.0/app/assets/stylesheets/jquery.ui.theme.css.erb'

Thank you for maintaining a great gem!

…ls::Helper::DependencyError), for pngs missing dependency declaration in ui theme.

Asset depends on 'jquery-ui/ui-bg_flat_75_ffffff_40x100.png' to generate properly but has not declared the dependency
Please add: `//= depend_on_asset "jquery-ui/ui-bg_flat_75_ffffff_40x100.png"` to '/Users/lu/.rvm/gems/ruby-2.0.0-p353@shiftit_rails3.2.16/gems/jquery-ui-rails-4.1.0/app/assets/stylesheets/jquery.ui.theme.css.erb'
@joliss
Copy link
Member

joliss commented Dec 12, 2013

Please see my response in #59 (comment). Let's continue the discussion there for now.

rake aborted!
"\xC3" on US-ASCII
/Users/lu/.rvm/gems/ruby-2.0.0-p0/gems/json-1.8.1/lib/json/common.rb:155:in `encode'
/Users/lu/.rvm/gems/ruby-2.0.0-p0/gems/json-1.8.1/lib/json/common.rb:155:in `initialize'
/Users/lu/.rvm/gems/ruby-2.0.0-p0/gems/json-1.8.1/lib/json/common.rb:155:in `new'
/Users/lu/.rvm/gems/ruby-2.0.0-p0/gems/json-1.8.1/lib/json/common.rb:155:in `parse'
/Users/lu/jquery-ui-rails/Rakefile:38:in `block in map_dependencies'
/Users/lu/jquery-ui-rails/Rakefile:37:in `each'
/Users/lu/jquery-ui-rails/Rakefile:37:in `map_dependencies'
/Users/lu/jquery-ui-rails/Rakefile:54:in `<top (required)>'
/Users/lu/.rvm/gems/ruby-2.0.0-p0/bin/ruby_noexec_wrapper:14:in `eval'
/Users/lu/.rvm/gems/ruby-2.0.0-p0/bin/ruby_noexec_wrapper:14:in `<main>'
@luisalima
Copy link
Collaborator Author

Hi @joliss , I modified my pull request to automatically generate the missing dependency declarations in the Rakefile.

Modified asset files are already generated using the new method.

I also added an encoding to the Rakefile since at least here (ruby2.0.0) it was giving a \xC3 on US-ASCII error.

joliss added a commit that referenced this pull request Dec 15, 2013
Fix missing dependency declarations (accused by sprockets_better_errors gem).
@joliss joliss merged commit 09668f5 into jquery-ui-rails:master Dec 15, 2013
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.

2 participants