Skip to content

Commit

Permalink
remove the env var for setting download concurrency (#2020)
Browse files Browse the repository at this point in the history
  • Loading branch information
KristofferC authored Sep 15, 2020
1 parent e6399b7 commit c2c9980
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 19 deletions.
4 changes: 1 addition & 3 deletions src/Types.jl
Original file line number Diff line number Diff line change
Expand Up @@ -326,9 +326,7 @@ Base.@kwdef mutable struct Context
io::IO = something(DEFAULT_IO[], stderr)
use_libgit2_for_all_downloads::Bool = false
use_only_tarballs_for_downloads::Bool = false
# NOTE: The JULIA_PKG_CONCURRENCY environment variable is likely to be removed in
# the future. It currently stands as an unofficial workaround for issue #795.
num_concurrent_downloads::Int = haskey(ENV, "JULIA_PKG_CONCURRENCY") ? parse(Int, ENV["JULIA_PKG_CONCURRENCY"]) : 8
num_concurrent_downloads::Int = 8
graph_verbose::Bool = false
currently_running_target::Bool = false
# test instrumenting
Expand Down
9 changes: 0 additions & 9 deletions test/new.jl
Original file line number Diff line number Diff line change
Expand Up @@ -2248,15 +2248,6 @@ end
end
end

@testset "Set download concurrency" begin
isolate() do
withenv("JULIA_PKG_CONCURRENCY" => 1) do
ctx = Pkg.Types.Context()
@test ctx.num_concurrent_downloads == 1
end
end
end

@testset "API details" begin
# API should not mutate
isolate() do
Expand Down
7 changes: 0 additions & 7 deletions test/pkg.jl
Original file line number Diff line number Diff line change
Expand Up @@ -526,13 +526,6 @@ end end
@test Pkg.Types.pathrepr(path) == "`@stdlib/Test`"
end

@testset "Set download concurrency" begin
withenv("JULIA_PKG_CONCURRENCY" => 1) do
ctx = Pkg.Types.Context()
@test ctx.num_concurrent_downloads == 1
end
end

@testset "stdlib_resolve!" begin
a = Pkg.Types.PackageSpec(name="Markdown")
b = Pkg.Types.PackageSpec(uuid=UUID("9abbd945-dff8-562f-b5e8-e1ebf5ef1b79"))
Expand Down

0 comments on commit c2c9980

Please sign in to comment.