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

Fix usage of Pkg internals for Julia >= 1.6.2, fixes #224. #229

Merged
merged 1 commit into from
Jul 16, 2021

Conversation

fredrikekre
Copy link
Member

No description provided.

@fredrikekre
Copy link
Member Author

Ah, of course: #228

I think this PR fixes usage of ctx.env.manifest in some more places though? Maybe not.

@pfitzseb
Copy link
Member

I'd be happy to go with this if you fix the 1.0 test failure.

@fredrikekre
Copy link
Member Author

fredrikekre commented Jul 16, 2021

This crashes Julia for me (but then passes tests...) on Julia 1.0.

Specifically:

signal (15): Terminated
in expression starting at /home/fredrik/dev/SymbolServer/src/utils.jl:460
apply_cl at /buildworker/worker/package_linux64/build/src/flisp/flisp.c:1678
do_trycatch at /buildworker/worker/package_linux64/build/src/flisp/flisp.c:898
apply_cl at /buildworker/worker/package_linux64/build/src/flisp/flisp.c:1806
_applyn at /buildworker/worker/package_linux64/build/src/flisp/flisp.c:682
fl_applyn at /buildworker/worker/package_linux64/build/src/flisp/flisp.c:722
jl_parse_eval_all at /buildworker/worker/package_linux64/build/src/ast.c:827
jl_load at /buildworker/worker/package_linux64/build/src/toplevel.c:839

so looks like something in

function get_file_from_cloud(manifest, uuid, environment_path, depot_dir, cache_dir = "../cache", download_dir = "../downloads/")

@fredrikekre
Copy link
Member Author

That also happens on SymbolServer master and on CI https://github.com/julia-vscode/SymbolServer.jl/pull/229/checks?check_run_id=3085321049#step:5:16 so probably not because of this PR?

@pfitzseb pfitzseb merged commit 1bb8c9e into julia-vscode:master Jul 16, 2021
@ZacLN
Copy link
Contributor

ZacLN commented Jul 23, 2021

This seems to break symbolserver on my machine:

ERROR: type Dict has no field deps
Stacktrace:
 [1] getproperty(x::Dict{Base.UUID, Pkg.Types.PackageEntry}, f::Symbol)
   @ Base ./Base.jl:33
 [2] read_manifest(manifest_filename::String)
   @ SymbolServer ~/.vscode/extensions/julialang.language-julia-insider-1.2.8/scripts/packages/SymbolServer/src/utils.jl:32
 [3] getstore(ssi::SymbolServerInstance, environment_path::String, progress_callback::LanguageServer.var"#93#95"{LanguageServerInstance}, error_handler::var"#3#4"; download::Bool)
   @ SymbolServer ~/.vscode/extensions/julialang.language-julia-insider-1.2.8/scripts/packages/SymbolServer/src/SymbolServer.jl:33
 [4] macro expansion
   @ ~/.vscode/extensions/julialang.language-julia-insider-1.2.8/scripts/packages/LanguageServer/src/languageserverinstance.jl:182 [inlined]
 [5] (::LanguageServer.var"#92#94"{LanguageServerInstance})()
   @ LanguageServer ./task.jl:411

@fredrikekre
Copy link
Member Author

Julia version?

@ZacLN
Copy link
Contributor

ZacLN commented Jul 23, 2021

1.7.0-DEV.922

@fredrikekre
Copy link
Member Author

Perhaps that is some intermediate version that doesn't have the Pkg updates. Do you also get it with a 1.7 beta?

@fredrikekre fredrikekre deleted the fe/1.6.2 branch July 23, 2021 19:05
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 this pull request may close these issues.

3 participants