-
Notifications
You must be signed in to change notification settings - Fork 493
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: update to 3.4.1 #5144
ruby: update to 3.4.1 #5144
Conversation
jeremyd2019
commented
Jan 16, 2025
•
edited
Loading
edited
- Update to 3.4.1
- remove unnecessary manual extraction of sources, the default works just fine
- patch syslog gem to build on cygwin (it builds fine)
- patch win32/resolv to include headers when checking for GetNetworkParams function (it otherwise fails to find it on i686, presumably due to stdcall)
- limit build's make to -j2, due to errors apparently due to exhaustion of page file space
Weirdnesses:
|
on i686 it looks like it's segfauling in |
Ugh, my patch to fix i686 broke x86_64 (and of course, details of what went wrong are not in the CI log, have to build locally to see what's going on). More work to do here |
141b364
to
860c3d0
Compare
it builds locally for me now for both i686 and x86_64 (and passes tests), but the build is still unreliable. It worked at |
I tried to build with
My build environment is
|
Hmm, I've had a lot of issues with the build phase, but check has generally been OK (I've seen a couple of spurious failures in fork/signal tests under load). |
I also tried |
I added a |
I tried with HEAD of your branch. So it means the freeze happened with |
I meant adding |
Got it. After applying following change
Agreed. I don't know the detail of |
I saw the same fixup_mmaps_after_fork issues with ruby 3.3.7, and with upstream cygwin. https://cygwin.com/pipermail/cygwin/2025-January/257155.html suggests it's due to pagefile size? |
a27a05b
to
73de5a0
Compare
patch to make win32/resolv build on i686
High -j results in errors in fixup_mmaps_after_fork apparently due to insufficient page file space. See https://cygwin.com/pipermail/cygwin/2025-January/257155.html
73de5a0
to
de6dc3d
Compare
I figure -j2 is a reasonable compromise to avoid pagefile exhaustion while still getting some parallelism in the build. Once (assuming) this is merged, I figure I should try to get my patches upstream, and report a bug for the extmk.rb error at least. |
There are no errors extracting the source, so just let the default extraction do its thing.
-j2 still fails an uncomfortable amount of the time on i686, but as that's unsupported, I can live with having to retry a few times there. |