Skip to content

Commit 0091cfe

Browse files
authored
Skip another test (the create_library() test) on Julia 1.12 (because it does incremental=false)
1 parent 0ede753 commit 0091cfe

File tree

1 file changed

+30
-17
lines changed

1 file changed

+30
-17
lines changed

test/runtests.jl

Lines changed: 30 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -88,9 +88,11 @@ end
8888
@testset for incremental in (is_slow_ci ? (false,) : (true, false))
8989
if incremental == false
9090
if is_julia_1_11 || is_julia_1_12
91-
# On Julia 1.11 and 1.12, `incremental=false` is currently broken: https://github.com/JuliaLang/PackageCompiler.jl/issues/976
91+
# On Julia 1.11 and 1.12, `incremental=false` is currently broken.
92+
# 1.11: https://github.com/JuliaLang/PackageCompiler.jl/issues/976
93+
# 1.12: No GitHub issue yet.
9294
# So, for now, we skip the `incremental=false` tests on Julia 1.11 and 1.12
93-
@warn "This is Julia $(VERSION.major).$(VERSION.minor); skipping incremental=false test due to known bug: https://github.com/JuliaLang/PackageCompiler.jl/issues/976"
95+
@warn "This is Julia $(VERSION.major).$(VERSION.minor); skipping incremental=false test due to known bug: #976 (for 1.11), issue TODO (for 1.12)"
9496
@test_skip false
9597
continue
9698
end
@@ -194,21 +196,32 @@ end
194196
end # testset
195197

196198
if !is_slow_ci
197-
# Test library creation
198-
lib_source_dir = joinpath(@__DIR__, "..", "examples/MyLib")
199-
lib_target_dir = joinpath(tmp, "MyLibCompiled")
200-
201-
incremental = false
202-
filter = true
203-
lib_name = "inc"
204-
205-
tmp_lib_src_dir = joinpath(tmp, "MyLib")
206-
cp(lib_source_dir, tmp_lib_src_dir)
207-
create_library(tmp_lib_src_dir, lib_target_dir; incremental=incremental, force=true, filter_stdlibs=filter,
208-
precompile_execution_file=joinpath(lib_source_dir, "build", "generate_precompile.jl"),
209-
precompile_statements_file=joinpath(lib_source_dir, "build", "additional_precompile.jl"),
210-
lib_name=lib_name, version=v"1.0.0")
211-
rm(tmp_lib_src_dir; recursive=true)
199+
if is_julia_1_12
200+
# On Julia 1.12, `incremental=false` is currently broken when doing `create_library()`.
201+
# 1.12: No GitHub issue yet.
202+
# So, for now, we skip the `incremental=false` tests on Julia 1.12 when doing `create_library()`.
203+
@warn "This is Julia $(VERSION.major).$(VERSION.minor); skipping incremental=false test when doing `create_library()` due to known bug: issue TODO (for 1.12)"
204+
@test_skip false
205+
continue
206+
else
207+
# Test library creation
208+
lib_source_dir = joinpath(@__DIR__, "..", "examples/MyLib")
209+
lib_target_dir = joinpath(tmp, "MyLibCompiled")
210+
211+
# This is why we have to skip this test on 1.12:
212+
incremental = false
213+
214+
filter = true
215+
lib_name = "inc"
216+
217+
tmp_lib_src_dir = joinpath(tmp, "MyLib")
218+
cp(lib_source_dir, tmp_lib_src_dir)
219+
create_library(tmp_lib_src_dir, lib_target_dir; incremental=incremental, force=true, filter_stdlibs=filter,
220+
precompile_execution_file=joinpath(lib_source_dir, "build", "generate_precompile.jl"),
221+
precompile_statements_file=joinpath(lib_source_dir, "build", "additional_precompile.jl"),
222+
lib_name=lib_name, version=v"1.0.0")
223+
rm(tmp_lib_src_dir; recursive=true)
224+
end
212225
end
213226

214227
# Test creating an empty sysimage

0 commit comments

Comments
 (0)