-
Notifications
You must be signed in to change notification settings - Fork 682
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
Fix uninitialized constant Parser::AST::Processor::Mixin #7030
Conversation
✅ Deploy Preview for chef-inspec canceled.
|
d459348
to
ad55c9a
Compare
ad55c9a
to
9defb88
Compare
parser 3.3.1.0 introduced whitequark/parser#1000, which causes this failure: ``` % bundle exec ruby -Itest test/unit/profiles/profile_test.rb inspec/lib/inspec/utils/profile_ast_helpers.rb:7:in `<class:CollectorBase>': uninitialized constant Parser::AST::Processor::Mixin (NameError) include Parser::AST::Processor::Mixin ^^^^^^^ ``` Fix this by inherting from `Parser::AST::Processor` instead. Closes inspec#7029 Signed-off-by: Stan Hu <stanhu@gmail.com>
9defb88
to
d2ff553
Compare
What's the current work around for existing releases, like 6.6.0? |
Use |
Aka throw that in the Gemfile of my profile? Working to fix my current pipeline while we wait on the patch. |
Yes, lock it down. 😄 |
Cool, I will give that a try, or more specifically @wdower will :) |
We tried this but without much success, any idea when this fix will be merged?
|
It would be merged today! |
parser 3.3.1.0 introduced whitequark/parser#1000, which causes this failure: ``` % bundle exec ruby -Itest test/unit/profiles/profile_test.rb inspec/lib/inspec/utils/profile_ast_helpers.rb:7:in `<class:CollectorBase>': uninitialized constant Parser::AST::Processor::Mixin (NameError) include Parser::AST::Processor::Mixin ^^^^^^^ ``` Fix this by inherting from `Parser::AST::Processor` instead. Closes #7029 Signed-off-by: Stan Hu <stanhu@gmail.com>
parser 3.3.1.0 introduced whitequark/parser#1000, which causes this failure: ``` % bundle exec ruby -Itest test/unit/profiles/profile_test.rb inspec/lib/inspec/utils/profile_ast_helpers.rb:7:in `<class:CollectorBase>': uninitialized constant Parser::AST::Processor::Mixin (NameError) include Parser::AST::Processor::Mixin ^^^^^^^ ``` Fix this by inherting from `Parser::AST::Processor` instead. Closes #7029 Signed-off-by: Stan Hu <stanhu@gmail.com> Co-authored-by: Stan Hu <stanhu@gmail.com>
It looks like Locking |
That works for me
…--------
Aaron Lippold
***@***.***
260-255-4779
twitter/aim/yahoo,etc.
'aaronlippold'
On Wed, May 1, 2024 at 16:10 Aaron Lippold ***@***.***> wrote:
Cool, I will give that a try, or more specifically @wdower
<https://github.com/wdower> will :)
—
Reply to this email directly, view it on GitHub
<#7030 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AALK42CJMJEERVNS2KAE3FDZAFD3VAVCNFSM6AAAAABG3RHCJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBZGA2TOOBQHA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
@ahasunos I see there is a v6.6.19 tag with this change (9939b0d), but https://rubygems.org/gems/inspec still only shows 6.6.0 published. Do you know what it takes to publish this to RubyGems? |
Thanks, that did the trick! |
Description
parser 3.3.1.0 introduced whitequark/parser#1000, which causes this failure:
Fix this by inherting from
Parser::AST::Processor
and requiring the right version of theparser
gem.Related Issue
Closes #7029
Types of changes
Checklist: