-
Notifications
You must be signed in to change notification settings - Fork 36
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
Remove unused jl_stdout_obj
import since it was removed in Julia 1.11
#237
base: master
Are you sure you want to change the base?
Conversation
Looking forward to get this merged! Currently RCall blocks me from transitioning to Julia 1.11 |
@JackDunnNZ thank you for you work. What is needed to get this merged? I am really blocked by this and would like to speed up the process if possible |
I believe we are all in wait of And a discussion on what to do next should happen there. Other PR like #210 are opened since more than 1 year with no reaction. |
bump. [My alternative suggestion is at JuliaLang, (partial) revert of the breaking change there.] If this wasn't actually used, can't this be speedily merged? And presumably solving the problem (on master; and then tag a new version?). |
Please read the issue linked above, @Non-Contradiction is unfortunately nowhere to be found |
I did and I realize @Non-Contradiction is absent, maybe only one(?) with merge access. I would be willing to fix this (get maintainer status), I see you @JackDunnNZ have some commits, presumably no access (nor any other contributor), why I also edited, mentioning my alternative at JulaLang. It's bad that 1.11 broke this (possibly it wasn't documented API), though understandable PkgEval didn't catch (and thus probably thought ok); see issue I linked and commented on. |
Yes, unfortunately I have no commit access - I am merely an invested user that fixes things each time they have broken :) |
This
jl_stdout_obj
doesn't seem to be used anywhere in the package, and was removed in Julia 1.11: JuliaLang/julia#53250Fixes #234