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

Library uses Ruby 2.3+ code #27

Closed
tas50 opened this issue Jun 9, 2018 · 2 comments
Closed

Library uses Ruby 2.3+ code #27

tas50 opened this issue Jun 9, 2018 · 2 comments
Assignees

Comments

@tas50
Copy link
Contributor

tas50 commented Jun 9, 2018

The library is currently using some Ruby 2.3+ code. This is fine for including in Chef client, but we can't use it with the windows cookbook at the moment since it supports Chef 12.7 or later which shipped with Ruby 2.0. We get the following errors running it on Chef 12.7.

       Running handlers:
       [2018-06-09T07:54:00-07:00] ERROR: Running exception handlers
       Running handlers complete
       [2018-06-09T07:54:00-07:00] ERROR: Exception handlers complete
       Chef Client failed. 9 resources updated in 18 seconds
       [2018-06-09T07:54:00-07:00] FATAL: Stacktrace dumped to C:/Users/vagrant/AppData/Local/Temp/kitchen/cache/chef-stacktrace.out
       [2018-06-09T07:54:00-07:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
       [2018-06-09T07:54:00-07:00] FATAL: SyntaxError: windows_certificate[C:/certs/GlobalSignRootCA.pem] (test::certificate line 35) had an error: SyntaxError: C:/opscode/chef/embedded/lib/ruby/gems/2.0.0/gems/win32-certstore-0.1.0/lib/win32/certstore/mixin/assertions.rb:25: syntax error, unexpected '.'
          unless valid_store_name.include?(store_name&.upcase)

                                                       ^
       C:/opscode/chef/embedded/lib/ruby/gems/2.0.0/gems/win32-certstore-0.1.0/lib/win32/certstore/mixin/assertions.rb:90: syntax error, unexpected keyword_end, expecting end-of-input```
@tas50
Copy link
Contributor Author

tas50 commented Jun 9, 2018

@piyushawasthi
Copy link
Contributor

Thanks @tas50, This issue fixed in PR: #32

@piyushawasthi piyushawasthi self-assigned this Jun 11, 2018
@tas50 tas50 closed this as completed in #32 Jun 11, 2018
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

2 participants