JuliaParallel / Dagger.jl

A framework for out-of-core and parallel execution
Other
610 stars 66 forks source link

Cannot precompile Dagger on Julia 1.11.0-beta1 #503

Closed eschnett closed 2 months ago

eschnett commented 2 months ago

I see the error below. It seems ScopedValue is not defined.

(@v1.11) pkg> update
    Updating registry at `~/.julia/registries/General.toml`
  No Changes to `~/.julia/environments/v1.11/Project.toml`
  No Changes to `~/.julia/environments/v1.11/Manifest.toml`
Precompiling project...
  ✗ Dagger
  0 dependencies successfully precompiled in 14 seconds. 691 already precompiled.
  1 dependency errored.
  For a report of the errors see `julia> err`. To retry use `pkg> precompile`

julia> err
PkgPrecompileError: The following 1 direct dependency failed to precompile:

Dagger

Failed to precompile Dagger [d58978e5-989f-55fb-8d15-ea34adc7bf54] to "/Users/eschnett/.julia/compiled/v1.11/Dagger/jl_jxQb20".
ERROR: LoadError: UndefVarError: `ScopedValue` not defined in `Dagger`
Stacktrace:
 [1] top-level scope
   @ ~/.julia/packages/Dagger/Tx54v/src/options.jl:3
 [2] include(mod::Module, _path::String)
   @ Base ./Base.jl:558
 [3] include(x::String)
   @ Dagger ~/.julia/packages/Dagger/Tx54v/src/Dagger.jl:1
 [4] top-level scope
   @ ~/.julia/packages/Dagger/Tx54v/src/Dagger.jl:29
 [5] include
   @ ./Base.jl:558 [inlined]
 [6] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt128}}, source::Nothing)
   @ Base ./loading.jl:2721
 [7] top-level scope
   @ stdin:4
in expression starting at /Users/eschnett/.julia/packages/Dagger/Tx54v/src/options.jl:3
in expression starting at /Users/eschnett/.julia/packages/Dagger/Tx54v/src/Dagger.jl:1
in expression starting at stdin:
jpsamaroo commented 2 months ago

I can't reproduce this on master - can you remove and re-add Dagger? I don't see any issue with how our ScopedValue usage is defined in the codebase that would cause this.

eschnett commented 2 months ago

For some reason I picked up Dagger 0.18.8 (and it precompiled fine...) even after Dagger 0.18.9 was released. With 0.18.9 all is fine now.