Skip to content

Commit 3b444f5

Browse files
use julia_version kwarg rather than ctx
1 parent c925b84 commit 3b444f5

File tree

2 files changed

+9
-12
lines changed

2 files changed

+9
-12
lines changed

src/Prefix.jl

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -597,7 +597,8 @@ function get_addable_spec(name::AbstractString, version::VersionNumber;
597597
uuid=uuid,
598598
#version=version,
599599
tree_hash=tree_hash_sha1,
600-
repo=Pkg.Types.GitRepo(rev=git_commit_sha, source=valid_url),
600+
rev=git_commit_sha,
601+
url=valid_url,
601602
)
602603
end
603604

@@ -669,18 +670,16 @@ function setup_dependencies(prefix::Prefix,
669670
deps_project = joinpath(prefix, triplet(platform), ".project")
670671
Pkg.activate(deps_project) do
671672
# Update registry first, in case the jll packages we're looking for have just been registered/updated
672-
ctx = Pkg.Types.Context(;julia_version)
673673
outs = verbose ? stdout : devnull
674674
update_registry(outs)
675675

676-
# Add all dependencies. Note: Pkg.add(ctx, deps) modifies in-place `deps` without
677-
# notice. We need to `deepcopy` the argument to prevent it from modying our
678-
# dependencies from under our feet: <https://github.com/JuliaLang/Pkg.jl/issues/3112>.
679-
Pkg_add(ctx, deepcopy(dependencies); platform=platform, io=outs)
676+
# Add all dependencies.
677+
Pkg_add(dependencies; platform=platform, io=outs, julia_version=julia_version)
680678

681679
# Ony Julia v1.6, `Pkg.add()` doesn't mutate `dependencies`, so we can't use the `UUID`
682680
# that was found during resolution there. Instead, we'll make use of `ctx.env` to figure
683681
# out the UUIDs of all our packages.
682+
ctx = Pkg.Types.Context()
684683
dependency_uuids = Set([uuid for (uuid, pkg) in ctx.env.manifest if pkg.name dependencies_names])
685684

686685
# Some JLLs are also standard libraries that may be present in the manifest because
@@ -715,10 +714,10 @@ function setup_dependencies(prefix::Prefix,
715714
push!(stdlib_pkgspecs, get_addable_spec(dep.name, dep.version; verbose))
716715
end
717716
end
718-
719717
# Re-install stdlib dependencies, but this time with `julia_version = nothing`
720718
if !isempty(stdlib_pkgspecs)
721-
Pkg_add(ctx, stdlib_pkgspecs; io=outs, julia_version=nothing)
719+
# TODO: shouldn't this take platform?
720+
Pkg_add(stdlib_pkgspecs; io=outs, julia_version=nothing)
722721
end
723722

724723
# Load their Artifacts.toml files

test/dependencies.jl

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -165,10 +165,8 @@ end
165165
uuid="86de99a1-58d6-5da7-8064-bd56ce2e322c",
166166
tree_hash=Base.SHA1("83481d62501cf2ef22bed745dbcedc4e75fa6e95"),
167167
version=PKG_VERSIONS.VersionSpec("*"),
168-
repo=Pkg.Types.GitRepo(
169-
source="https://github.com/JuliaBinaryWrappers/LLVM_jll.jl.git",
170-
rev="2772761b330d51146ace3125b26acdad0df4f30f",
171-
),
168+
url="https://github.com/JuliaBinaryWrappers/LLVM_jll.jl.git",
169+
rev="2772761b330d51146ace3125b26acdad0df4f30f",
172170
)
173171

174172
with_temp_project() do dir

0 commit comments

Comments
 (0)