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

Package stopped working after update to chefdk 2.0.28 #9

Closed
lucasslima opened this issue Jul 26, 2017 · 3 comments
Closed

Package stopped working after update to chefdk 2.0.28 #9

lucasslima opened this issue Jul 26, 2017 · 3 comments

Comments

@lucasslima
Copy link

lucasslima commented Jul 26, 2017

Hi,

The package is throwing errors after I updated my chefdk to 2.0.28.

This is the message I get:

[Linter] Error running cookstyle Error: -s/--stdin requires exactly one path.
/opt/chefdk/embedded/lib/ruby/gems/2.4.0/gems/rubocop-0.49.1/lib/rubocop/options.rb:30:in `parse'
/opt/chefdk/embedded/lib/ruby/gems/2.4.0/gems/rubocop-0.49.1/lib/rubocop/cli.rb:23:in `run'
/opt/chefdk/embedded/lib/ruby/gems/2.4.0/gems/rubocop-0.49.1/bin/rubocop:13:in `block in <top (required)>'
/opt/chefdk/embedded/lib/ruby/2.4.0/benchmark.rb:308:in `realtime'
/opt/chefdk/embedded/lib/ruby/gems/2.4.0/gems/rubocop-0.49.1/bin/rubocop:12:in `<top (required)>'
/opt/chefdk/embedded/lib/ruby/gems/2.4.0/gems/cookstyle-2.0.0/bin/cookstyle:13:in `load'
/opt/chefdk/embedded/lib/ruby/gems/2.4.0/gems/cookstyle-2.0.0/bin/cookstyle:13:in `<top (required)>'
/opt/chefdk/bin/cookstyle:263:in `load'
/opt/chefdk/bin/cookstyle:263:in `<main>'
    at /home/lucas/.atom/packages/linter-cookstyle/lib/index.coffee:48:15

Any way i can fix this?

@nweddle
Copy link
Contributor

nweddle commented Aug 4, 2017

Looks like this is caused by the version of rubocop (0.49.1) that was included in ChefDK 2.0.13
See: rubocop/rubocop#4226

nweddle added a commit to nweddle/linter-cookstyle that referenced this issue Aug 4, 2017
mattstratton added a commit that referenced this issue Aug 5, 2017
Move stdin argument to end of exec command, fixes #9
@mattstratton
Copy link
Owner

This has been published to the package repository, so feel free to update!

@lucasslima
Copy link
Author

Awesome! Thanks for the quick resolution!

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