-
-
Notifications
You must be signed in to change notification settings - Fork 270
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
Change semantics of respect_sysimage_versions
#3627
Comments
The point here is that you cannot load any other Tar than the one in the sysimage so doing a |
Right. So not a regression because in 1.6 this seemed to work at first, but didn't actually work (@v1.6) pkg> dev Tar
Resolving package versions...
Updating `~/.julia/environments/v1.6/Project.toml`
[a4e569a6] + Tar `~/.julia/dev/Tar`
No Changes to `~/.julia/environments/v1.6/Manifest.toml`
julia> using Tar
julia> @which Tar.read_data(stdout, stdout)
read_data(tar::IO, file::IO; size, buf, tee) in Tar at /Users/x/.julia/juliaup/julia-1.6.7+0.x64.apple.darwin14/share/julia/stdlib/v1.6/Tar/src/extract.jl:642 To make this feature request reasonable, we would need a way of loading a version of Tar other than the one in the syimg. (something like, but ideally more pretty than the UUID hack we currently recommend for |
The solution in my opinion is to move out stdlibs from the sysimage which is very much an active work in progress. |
Most stdlibs are now out of the sysimage so |
I think
]dev Tar
should continue to "just work".I propose to change the semantics to either
A (preferred)
false => no change (syimg versions are still irrelevant)
true => force using sysimg versions for anything that is not explicitly set in Project.toml.
or B (less breaking, but this fixes a regression so idk if that's really an issue)
false => no change (syimg versions are still irrelevant)
:auto => use force using sysimg versions for anything that is not explicitly set in Project.toml.
true => no change (force use sysimg versions everywhere)
See also: JuliaLang/julia#51374, #3002, cc @KristofferC
The text was updated successfully, but these errors were encountered: