Skip to content

Commit

Permalink
Revert "do not intentionally suppress errors in precompile script fro…
Browse files Browse the repository at this point in the history
…m being …"

This reverts commit 4a4ca9c.
  • Loading branch information
KristofferC authored Oct 9, 2024
1 parent ecf41b1 commit d65f48a
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions contrib/generate_precompile.jl
Original file line number Diff line number Diff line change
Expand Up @@ -347,7 +347,8 @@ generate_precompile_statements() = try # Make sure `ansi_enablecursor` is printe
print_state("step1" => "F$n_step1")
return :ok
end
PARALLEL_PRECOMPILATION ? bind(statements_step1, step1) : wait(step1)
Base.errormonitor(step1)
!PARALLEL_PRECOMPILATION && wait(step1)

# Create a staging area where all the loaded packages are available
PrecompileStagingArea = Module()
Expand All @@ -361,7 +362,7 @@ generate_precompile_statements() = try # Make sure `ansi_enablecursor` is printe
# Make statements unique
statements = Set{String}()
# Execute the precompile statements
for statement in statements_step1
for sts in [statements_step1,], statement in sts
# Main should be completely clean
occursin("Main.", statement) && continue
Base.in!(statement, statements) && continue
Expand Down Expand Up @@ -397,7 +398,6 @@ generate_precompile_statements() = try # Make sure `ansi_enablecursor` is printe
println()
# Seems like a reasonable number right now, adjust as needed
# comment out if debugging script
have_repl = false
n_succeeded > (have_repl ? 650 : 90) || @warn "Only $n_succeeded precompile statements"

fetch(step1) == :ok || throw("Step 1 of collecting precompiles failed.")
Expand All @@ -408,6 +408,7 @@ generate_precompile_statements() = try # Make sure `ansi_enablecursor` is printe
finally
fancyprint && print(ansi_enablecursor)
GC.gc(true); GC.gc(false); # reduce memory footprint
return
end

generate_precompile_statements()
Expand Down

0 comments on commit d65f48a

Please sign in to comment.