Skip to content

Commit b910d5e

Browse files
committed
Prune LinearAlgebra module in ambiguity test (#1349)
The ambiguity check was being run in a separate process, but it was not pruning the `LinearAlgebra` module. This was therefore not testing for ambiguities in the latest code, but the one in the sysimg. The PR would be able to replicate the test failure seen in https://buildkite.com/julialang/julia-master/builds/47424#0196c03a-2230-4d64-b404-e04dbfc76c3f (cherry picked from commit 0b46d5c)
1 parent fda7486 commit b910d5e

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

test/ambiguous_exec.jl

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,9 @@
11
# This file is a part of Julia. License is MIT: https://julialang.org/license
22

3+
module TestAmbiguity
4+
5+
isdefined(Main, :pruned_old_LA) || @eval Main include("prune_old_LA.jl")
6+
37
using Test, LinearAlgebra
48
let ambig = detect_ambiguities(LinearAlgebra; recursive=true)
59
@test isempty(ambig)
@@ -19,3 +23,5 @@ let ambig = detect_ambiguities(LinearAlgebra; recursive=true)
1923
@test isempty(expect)
2024
@test good
2125
end
26+
27+
end # module

0 commit comments

Comments
 (0)