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

Patch of Dierckx_jll causing errors in Dierckx.jl #10266

Closed
bclyons12 opened this issue Jan 14, 2025 · 6 comments · Fixed by #10268
Closed

Patch of Dierckx_jll causing errors in Dierckx.jl #10266

bclyons12 opened this issue Jan 14, 2025 · 6 comments · Fixed by #10268

Comments

@bclyons12
Copy link

Dierckx_jll recently release v0.1.1. As a patch release, that should be non-breaking, but there are several errors occurring in Dierckx.jl that are fixed by downgrading Dierckx_jll back to v0.1.0:
JuliaMath/Dierckx.jl#98
JuliaMath/Dierckx.jl#99

@giordano
Copy link
Member

Well, it's hard to predict that an upstream non-breaking version would break things, isn't it?

@imciner2
Copy link
Member

There isn't much we can do in Yggdrasil about this. You have two options basically: work with upstream Dierckx to fix the problem and get a new version we can build, or introduce a compat entry for the jll to filter out the broken version in the Dierckx.jl package.

@giordano
Copy link
Member

@bclyons12
Copy link
Author

If there were argument changes, that's a breaking change. Shouldn't this have been a v0.x minor release instead of a patch? If this were v0.2, it would have required people to check for compat instead of automatically updating.

@giordano
Copy link
Member

Well, it's hard to predict that an upstream non-breaking version would break things, isn't it?

👆

@imciner2
Copy link
Member

I guess the scipy people don't consider that to be public since it is their underlying Fortran code. The other option is going to the horses mouth and just getting stuff from netlib directly, but they don't seem to have an all-encompasing tar file for the dierckx library, just individual routines.

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 a pull request may close this issue.

3 participants