Skip to content

Commit 6fab34c

Browse files
Robert Marshallderiamis
andcommitted
Merge branch 'pin-parser-version-in-chef-bin' into 'master'
Pin parser dependency in chef-bin See merge request https://gitlab.com/gitlab-org/omnibus-gitlab/-/merge_requests/7572 Merged-by: Robert Marshall <[email protected]> Approved-by: Robert Marshall <[email protected]> Co-authored-by: Ryan Egesdahl <[email protected]>
2 parents 32ad31d + bc8d378 commit 6fab34c

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

config/software/chef-bin.rb

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,17 @@
3232
env = with_standard_compiler_flags(with_embedded_path)
3333
patch source: 'add-license-file.patch'
3434

35+
# TODO: rubocop-ast installs a later version of parser, which omits the AST::Processor
36+
# mixin, causing breakage. This is a temporary workaround until a more permanent fix
37+
# is in place: https://gitlab.com/gitlab-org/omnibus-gitlab/-/merge_requests/7362
38+
gem 'install parser' \
39+
" --clear-sources" \
40+
" -s https://packagecloud.io/cinc-project/stable" \
41+
" -s https://rubygems.org" \
42+
" --version '3.3.0.5'" \
43+
" --bindir '#{install_dir}/embedded/bin'" \
44+
' --no-document', env: env
45+
3546
# Temporary workaround because upstream inspec-core does not list this as
3647
# a requirement and it causes failures during gitlab-ctl reconfigure in
3748
# the QA job pipelines

0 commit comments

Comments
 (0)