-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
store the path to where a package got loaded #37586
Conversation
So cool! Is #35663 next? |
What behavior do you want there? A warning when the package would have gotten loaded from a different place than if it was not already loaded? |
Personally, I want an error---don't let the user change environments. |
Storing the path here is an improvement but it isn't really perfect. For example, if I make a sysimage, send it to a friend and they uses it, loads the same version of a package that is in the sysimage, that will have a different computed path where that package would be loaded from, but it is still the same package version. What we should store is probably more akin to "how this package got loaded". And there are a few ways a package can get loaded:
In the case above, the same |
Seems like this must have missed a case: julia> using JuliaInterpreter
julia> pathof(JuliaInterpreter)
"/home/tim/.julia/packages/JuliaInterpreter/muncv/src/JuliaInterpreter.jl"
julia> exit()
tim@diva:~/.julia/dev/Revise$ julia-master --startup-file=no -q
julia> using Revise
julia> using JuliaInterpreter
julia> pathof(JuliaInterpreter)
julia> |
Thanks, will look at it. |
And use it in
pkgdir
Fixes #33210