Skip to content

Conversation

@KristofferC
Copy link
Member

To be merged if JuliaLang/Pkg.jl#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 KristofferC added triage This should be discussed on a triage call backport 1.13 Change should be backported to release-1.13 labels Nov 5, 2025
@KristofferC KristofferC force-pushed the kc/no_checkbounds_stdlib_pkgimage branch from dcae480 to 3d7768b Compare November 12, 2025 22:19
@KristofferC
Copy link
Member Author

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

on CI. Wat?

@jakobnissen
Copy link
Member

@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]

@KristofferC
Copy link
Member Author

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

@LilithHafner
Copy link
Member

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.

@KristofferC KristofferC mentioned this pull request Jan 9, 2026
43 tasks
@DilumAluthge DilumAluthge removed the triage This should be discussed on a triage call label Feb 1, 2026
@KristofferC KristofferC mentioned this pull request Feb 4, 2026
21 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 1.13 Change should be backported to release-1.13

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants