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

Assets precompile error #435

Closed
Tamiyadd opened this issue May 26, 2016 · 5 comments
Closed

Assets precompile error #435

Tamiyadd opened this issue May 26, 2016 · 5 comments

Comments

@Tamiyadd
Copy link

Hi @justin808, i upgraded my react on rails version from 6.0.0.rc1 to 6.0.0 but i started to get an error on assets precompile.

When precompiling assets it throws the following error:

TypeError: no implicit conversion of nil into String
.rvm/gems/ruby-2.1.2/gems/react_on_rails-6.0.0/lib/tasks/assets.rake:30:in `initialize'
.rvm/gems/ruby-2.1.2/gems/react_on_rails-6.0.0/lib/tasks/assets.rake:30:in `new'
.rvm/gems/ruby-2.1.2/gems/react_on_rails-6.0.0/lib/tasks/assets.rake:30:in `block (3 levels) in <top (required)>'
.rvm/gems/ruby-2.1.2/gems/react_on_rails-6.0.0/lib/tasks/assets.rake:109:in `block in <top (required)>'
.rvm/gems/ruby-2.1.2/bin/ruby_executable_hooks:15:in `eval'
.rvm/gems/ruby-2.1.2/bin/ruby_executable_hooks:15:in `<main>'
Tasks: TOP => react_on_rails:assets:symlink_non_digested_assets
(See full trace by running task with --trace)

i investigated in that line of code and found that the gem search for .sprockets-manifest-*.json manifest file but i have a manifest-*.json manifest file.
Maybe the problem is there? i don't know why with the rc1 everything works well and with this version it's not working

can you help me?

thanks

@justin808
Copy link
Member

@alleycat-at-git @Tamiyadd Ahh! this must be related to the sprockets version.

We need to check for that and maybe allow either file.

Let me know on the sprockets version and we'll try to get this fixed asap. Also, can you post the contests of your manifest-*.json file?

@alexeuler
Copy link
Contributor

@Tamiyadd, could you please try this PR out? Did it fix the issue?

@Tamiyadd
Copy link
Author

@alleycat-at-git i tried your PR and it works like a charm!

for your info i'm using sprockets 2.12.4,
do you need my manifest.json content anyway?

@alexeuler
Copy link
Contributor

Cool! I think we now that you checked it worked, we don't need the file anymore.

On 27 May 2016, at 11:35, Tamiyadd notifications@github.com wrote:

@alleycat-at-git i tried your PR and it works like a charm!

for your info i'm using sprockets 2.12.4,
do you need my manifest.json content anyway?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@justin808
Copy link
Member

Released in 6.0.1.

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

No branches or pull requests

3 participants