-
-
Notifications
You must be signed in to change notification settings - Fork 85
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
Won't build with Ruby head #281
Comments
Ah, if I had to guess, it's because @ianks Does that sound right to you? (Note that this is not me asking for 3.4/head support, just wondering if that's a plausible reason for this.) |
I just 'copied' the file Your guess is probably correct. |
Things work perfectly with ruby-head, but you need to enable the |
Got it, thank you! For reference here’s the source PR: oxidize-rb/rb-sys#229 @MSP-Greg see if that works out for you. To be honest, I don’t really feel comfortable officially supporting Closing this out because I like to keep my issues clean, but we can keep discussing. |
@gjtorikian I'd encourage testing against ruby-head to help ruby core develop more easily. We run ruby-head in prod and anytime an extension fails to compile on ruby head, there's a lot of disruption. FWIW, |
Ok, you convinced me! Reopening since there's a task here. I'll take a PR for this, as I haven't got time to investigate the work. But I'll get to it after I finish #276. |
JFYI, I quickly checked today using Or, with Windows locally, it may be some squirrelly build tool issue... More time later in the week. |
@MSP-Greg I haven’t had time to look into this, but ruby-head has been failing on Windows for some time now: https://github.com/gjtorikian/commonmarker/actions/runs/11912463542/job/33196103377 Would you happen to have encountered this before? Looks like it can’t find the header file for |
Sorry for dropping this. I'll get back to this, but it may be a few days. Re CI and head builds, one might say "we're not in Kansas anymore". Almost all CI head builds are done running the full Ruby test suite, and if there are any failures/errors, the build doesn't replace the previous build. Also, there is better testing of the actual code created by Again, sorry, Greg |
No worries! Just curious if something rang a bell. |
I've got some code that runs regularly, and I have run it for years with Windows Ruby head (ucrt). I can't build Commonmarker v1.
I then added it to CI (test.yml), and it failed to build on all three platforms. See:
https://github.com/MSP-Greg/commonmarker/actions/runs/8402231998
Not sure if it's considered an issue. I'm not familiar with Rust, so I'm not sure whether it's a Ruby issue or a Commonmarker issue.
The text was updated successfully, but these errors were encountered: