-
Notifications
You must be signed in to change notification settings - Fork 334
Update to 4.0.0-preview2 #517
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
Conversation
3.5 is now 4.0: ruby/ruby@6d81969
|
lol, lmao, great; it lights up like the Christmas tree 4.0 will eventually be |
For |
|
Close + reopen to pick up docker-library/official-images#20310 in CI |
|
Do the test containers have git available (no idea how this gets run)? If that is the case, you can temporarily pin to the repository, that should get this green then: # Pinned to commit to allow bundler 4. Revert once https://github.com/rubysec/bundler-audit/pull/409 is released
gem 'bundler-audit', git: "https://github.com/rubysec/bundler-audit", ref: "309d20356a7ea435d0f92982b6fcb2a4bd57db66" |
|
Heh, yeah, the rabbit hole goes deeper though, unfortunately! Here's what I'm at right now: diff --git a/test/tests/ruby-binstubs/Gemfile b/test/tests/ruby-binstubs/Gemfile
index dd9516655..5760efb93 100644
--- a/test/tests/ruby-binstubs/Gemfile
+++ b/test/tests/ruby-binstubs/Gemfile
@@ -1,3 +1,8 @@
source 'https://rubygems.org'
-gem 'bundler-audit', '0.9.1'
+
+# https://github.com/rubysec/bundler-audit/pull/409
+# TODO drop this all back down to just "gem 'bundler-audit', '0.9.3'" once it's released: https://github.com/rubysec/bundler-audit/releases
+gem 'rubygems-tasks', github: 'postmodern/rubygems-tasks', ref: '8a080d9a730218e957bb8a1c9ab72f025c61d229' # https://github.com/postmodern/rubygems-tasks/pull/28 (this is a dep of bundler-audit)
+gem 'bundler-audit', github: 'rubysec/bundler-audit', ref: '309d20356a7ea435d0f92982b6fcb2a4bd57db66'
+
gem 'brakeman', '5.4.1'and it fails like this, after /usr/local/lib/ruby/4.0.0+0/rubygems.rb:263:in 'Gem.find_spec_for_exe': can't find gem bundler-audit (>= 0.a) with executable bundler-audit (Gem::GemNotFoundException)
from /usr/local/lib/ruby/4.0.0+0/rubygems.rb:237:in 'Gem.find_and_activate_spec_for_exe'
from /usr/local/lib/ruby/4.0.0+0/rubygems.rb:282:in 'Gem.activate_and_load_bin_path'
from /usr/local/bundle/bin/bundler-audit:25:in '<main>'(adding |
|
Hm, it works locally for me. How are you testing in the container? I didn't really figure that one out. |
|
Our test harness effectively does |
|
It's really driving me nuts; Use `bundle info [gemname]` to see where a bundled gem is installed.
* bundler-audit (0.9.2 309d203)
Summary: Patch-level verification for Bundler
Homepage: https://github.com/rubysec/bundler-audit#readme
Documentation: https://rubydoc.info/gems/bundler-audit
Source Code: https://github.com/rubysec/bundler-audit
Changelog: https://github.com/rubysec/bundler-audit/blob/master/ChangeLog.md
Bug Tracker: https://github.com/rubysec/bundler-audit/issues
Path: /usr/local/bundle/bundler/gems/bundler-audit-309d20356a7e |
|
Oh, this might just not work as a git gem. I already had Seems irrelevant actually which version is in the gemfile, it always uses the latest available version. So the gemfile is just there to install the gems, and for I guess this is ruby/rubygems#6957 or one of the related PRs/Issues |
|
I've opened docker-library/official-images#20318 with a different potential solution that hopefully still preserves the intent of the test -- I'm running it against all current tags of |
|
Nice, I looked for a different one but no luck. I think this is a good replacement, it adds the command in exactly the same way |
Changes: - docker-library/ruby@8416c3f: Merge pull request docker-library/ruby#517 from infosiftr/4.0 - docker-library/ruby@7e8d616: Update to 4.0.0-preview2
Changes: - docker-library/ruby@8416c3f: Merge pull request docker-library/ruby#517 from infosiftr/4.0 - docker-library/ruby@7e8d616: Update to 4.0.0-preview2 Co-authored-by: Docker Library Bot <doi+docker-library-bot@docker.com>
3.5 is now 4.0: ruby/ruby@6d81969