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

Remove support for deprecated Julia REQUIRE, Julia < 1.3 #1350

Merged
merged 3 commits into from
May 29, 2024

Conversation

minrk
Copy link
Member

@minrk minrk commented May 27, 2024

it doesn't work anymore because build dependencies of the old ZMQ.jl have been removed, so I don't see a feasible way for us to keep it working.

xref JuliaInterop/ZMQ.jl#228

closes #1349

@minrk minrk added the breaking label May 27, 2024
@minrk
Copy link
Member Author

minrk commented May 27, 2024

Looks like the oldest version of ZMQ.jl that doesn't rely on the removed repo requires Julia 1.3, starting with JuliaInterop/ZMQ.jl#194 . So now Julia 1.3 is the oldest version we can support.

it doesn't work anymore because build dependencies of the old ZMQ.jl have been removed.
@minrk minrk changed the title Remove support for deprecated Julia REQUIRE Remove support for deprecated Julia REQUIRE, Julia < 1.3 May 27, 2024
@minrk
Copy link
Member Author

minrk commented May 27, 2024

fwiw, julia 1.3 is from November 2019, about 4.5 years ago, and the newest version we supported with REQUIRE was 1.1.1, just over 5 years ago now.

@minrk minrk mentioned this pull request May 27, 2024
@minrk
Copy link
Member Author

minrk commented May 28, 2024

in fact, 1.3-1.5 fail with "No Such Device" in the precompile bind here:

ZMQ.bind(s1, "tcp://localhost:*")

so as it is now, only Julia >= 1.6 will actually work.

The precompile on 1.3-1.5 should be fixed by JuliaInterop/ZMQ.jl#237

@minrk
Copy link
Member Author

minrk commented May 28, 2024

It would probably behoove us to have frozen base environments for Julia eventually. That wouldn't nave solved the current problem, though.

Copy link
Member

@manics manics left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! https://github.com/JuliaInterop/ZMQ.jl/releases/tag/v1.2.5 was released, so if CI passes this can be merged

@manics manics merged commit fef3250 into jupyterhub:main May 29, 2024
18 checks passed
@yuvipanda yuvipanda added the bug label Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Julia tests are failing
3 participants