julia icon indicating copy to clipboard operation
julia copied to clipboard

no longer bundle pkgimages for stdlibs with forced bounds checking since Pkg.test no longer defaults to this

Open KristofferC opened this issue 1 month ago • 4 comments

To be merged if https://github.com/JuliaLang/Pkg.jl/pull/4494 is merged. Triage might have fun with this one. To be decided for 1.13.

For people that want to test with forced bounds check on e.g. CI they can use the cache artifact functionality to cache those package images.

KristofferC avatar Nov 05 '25 20:11 KristofferC

  BoundsError: attempt to access 3-element Vector{String} at index [1]

on CI. Wat?

KristofferC avatar Nov 13 '25 13:11 KristofferC

@KristofferC That exception almost certainly happens because some test uses try/catch to mutate some variable, then resets it before the error is printed.

Example:

julia> old_depot = copy(DEPOT_PATH)
       try
           empty!(DEPOT_PATH)
           DEPOT_PATH[1]
       finally
           copy!(DEPOT_PATH, old_depot)
       end
ERROR: BoundsError: attempt to access 3-element Vector{String} at index [1]

jakobnissen avatar Nov 28 '25 12:11 jakobnissen

Yep, I think the test that fails is not really valid anymore with this change.

KristofferC avatar Nov 28 '25 12:11 KristofferC

Triage wants to know how this impacts the runtime of julia-runtest and how many MB it saves (Sam mentioned it saves about 200MB out of a total of 1.1GB install size). It's hard to give an answer without real numbers here.

Another option is to have a different tarball for CI than regular users.

LilithHafner avatar Dec 04 '25 18:12 LilithHafner