Skip to content
Snippets Groups Projects
Commit 08aa039e604d authored by Ryan Egesdahl's avatar Ryan Egesdahl
Browse files

Pin parser dependency in chef-bin

Installing chef-bin via `gem install` makes all of its dependencies floating,
including the `parser` gem. Because the `parser` gem is *also* used by other
Ruby modules, the floating dependencies can cause failures, such as a recent
problem that caused `gitlab-ctl` to fail because `inspec-core` depends on
`parser` as well. This is a temporary fix to correct the floating dependency
on `parser` until a more permanent fix is in place.
parent fa97f598e893
No related branches found
No related tags found
2 merge requests!144heptapod#1622: making 17.0 the oldstable,!127Merge upstream Omnibus GitLab into Omnibus Heptapod
......@@ -32,6 +32,17 @@
env = with_standard_compiler_flags(with_embedded_path)
patch source: 'add-license-file.patch'
# TODO: rubocop-ast installs a later version of parser, which omits the AST::Processor
# mixin, causing breakage. This is a temporary workaround until a more permanent fix
# is in place: https://gitlab.com/gitlab-org/omnibus-gitlab/-/merge_requests/7362
gem 'install parser' \
" --clear-sources" \
" -s https://packagecloud.io/cinc-project/stable" \
" -s https://rubygems.org" \
" --version '3.3.0.5'" \
" --bindir '#{install_dir}/embedded/bin'" \
' --no-document', env: env
# Temporary workaround because upstream inspec-core does not list this as
# a requirement and it causes failures during gitlab-ctl reconfigure in
# the QA job pipelines
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment