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

contrib/build_sysimg.jl broken? #25695

Closed
quinnj opened this issue Jan 23, 2018 · 6 comments
Closed

contrib/build_sysimg.jl broken? #25695

quinnj opened this issue Jan 23, 2018 · 6 comments

Comments

@quinnj
Copy link
Member

quinnj commented Jan 23, 2018

docs/Docs.jl
LoadError("sysimg.jl", 567, LoadError("/Users/jacobquinn/julia/base/userimg.jl", 1, ArgumentError("Module A not found in current path.\nRun `Pkg.add(\"A\")` to install the A package.")))
ERROR: LoadError: failed process: Process(`/Users/jacobquinn/julia/usr/bin/julia -C native --output-ji nothing.ji --output-o nothing.o -J basecompiler.ji --startup-file=no sysimg.jl`, ProcessExited(1)) [1]
Stacktrace:
 [1] error(::String, ::Base.Process, ::String, ::Int64, ::String) at ./error.jl:42
 [2] pipeline_error(::Base.Process) at ./process.jl:694
 [3] run(::Cmd) at ./process.jl:656
 [4] (::getfield(, Symbol("##2#3")){Bool,String})() at /Users/jacobquinn/julia/contrib/build_sysimg.jl:81
 [5] cd(::getfield(, Symbol("##2#3")){Bool,String}, ::String) at ./file.jl:70
 [6] #build_sysimg#1(::Bool, ::Bool, ::Function, ::String, ::String, ::String) at /Users/jacobquinn/julia/contrib/build_sysimg.jl:48
 [7] (::getfield(, Symbol("#kw##build_sysimg")))(::NamedTuple{(:force, :debug),Tuple{Bool,Bool}}, ::typeof(build_sysimg), ::String, ::String, ::String) at ./<missing>:0
 [8] top-level scope at /Users/jacobquinn/julia/contrib/build_sysimg.jl:208
 [9] include at ./boot.jl:292 [inlined]
 [10] include_relative(::Module, ::String) at ./loading.jl:1011
 [11] include(::Module, ::String) at ./sysimg.jl:26
 [12] process_options(::Base.JLOptions) at ./client.jl:334
 [13] _start() at ./client.jl:386
in expression starting at /Users/jacobquinn/julia/contrib/build_sysimg.jl:208

might be same/similar problem to #25664?

@iblislin
Copy link
Member

Because of this?

julia/base/sysimg.jl

Lines 564 to 567 in 7756d0e

empty!(DEPOT_PATH)
empty!(LOAD_PATH)
Base.isfile("userimg.jl") && Base.include(Main, "userimg.jl")

LOAD_PATH got emptied.

@iblislin
Copy link
Member

I removed these empty! and it built.

@StefanKarpinski
Copy link
Member

That was in there before. It seems necessary to avoid double filling them in a normal build.

@iblislin
Copy link
Member

From git blame: 4a6e64d#diff-9a4126dd8386dbc7e2eb93f7bef02d1bR460

maybe we can just switch the order of include("userimg.jl").

@StefanKarpinski
Copy link
Member

cc @JeffBezanson, introduced along with the stdlib mechanism.

@KristofferC
Copy link
Member

build_sysimg.jl has been deleted.

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

No branches or pull requests

4 participants