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

[ruby] Build with base ruby 3.0 to bootstrap from git sources #11620

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kateinoigakukun
Copy link

@kateinoigakukun kateinoigakukun commented Feb 20, 2024

We recently updated the minimum required base ruby version to 3.0 (ruby/ruby#9976). This change updates the Dockerfile to build base ruby from tarball and install it instead of using Ubuntu focal's ruby package, which is still 2.7.

This fixes the latest build failure https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=66036#c5

Copy link

kateinoigakukun is either the primary contact or is in the CCs list of projects/ruby.
kateinoigakukun is a new contributor to projects/ruby. The PR must be approved by known contributors before it can be merged. The past contributors are: DonggeLiu, kevinbackhouse, mame, jhawthorn

We recently updated the minimum required base ruby version to 3.0
(ruby/ruby#9976). This change updates the
Dockerfile to build base ruby from tarball and install it instead of
using Ubuntu focal's ruby package, which is still 2.7.
@kateinoigakukun
Copy link
Author

Could you review this? @jhawthorn

@kateinoigakukun
Copy link
Author

kateinoigakukun commented Feb 20, 2024

Hmm, I have no idea on what's happening in the failures 🤔

@DavidKorczynski
Copy link
Collaborator

It looks like the fuzz_ruby_gems is running into an issue in one of the first iterations and this is blocking the CI. Could you try and run this fuzzer locally and confirm it works as it should?

@kateinoigakukun
Copy link
Author

kateinoigakukun commented Feb 26, 2024

Right, it's reproducible on my local machine but unfortunately I'm not sure how to address the issue and don't have enough time to have closer look. #11622

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

Successfully merging this pull request may close these issues.

None yet

3 participants