Update docstrings for io kwargs, some io kwarg fixes (#2402)

(cherry picked from commit 6e8b621)

Author: IanButterworth

src/API.jl

@@ -261,7 +261,7 @@ function up(ctx::Context, pkgs::Vector{PackageSpec};
     return
 end
 
-resolve(; kwargs...) = resolve(Context(); kwargs...)
+resolve(; io::IO=DEFAULT_IO[], kwargs...) = resolve(Context(;io); kwargs...)
 function resolve(ctx::Context; kwargs...)
     up(ctx; level=UPLEVEL_FIXED, mode=PKGMODE_MANIFEST, update_registry=false, kwargs...)
     return nothing
@@ -1390,7 +1390,7 @@ function instantiate(ctx::Context; manifest::Union{Bool, Nothing}=nothing,
     # Run build scripts
     Operations.build_versions(ctx, union(UUID[pkg.uuid for pkg in new_apply], new_git); verbose)
 
-    allow_autoprecomp && Pkg._auto_precompile(ctx; kwargs...)
+    allow_autoprecomp && Pkg._auto_precompile(ctx)
 end
 
 
@@ -1404,7 +1404,7 @@ function status(ctx::Context, pkgs::Vector{PackageSpec}; diff::Bool=false, mode=
 end
 
 
-function activate(;temp=false,shared=false, io::IO=DEFAULT_IO[])
+function activate(;temp=false, shared=false, io::IO=DEFAULT_IO[])
     shared && pkgerror("Must give a name for a shared environment")
     temp && return activate(mktempdir())
     Base.ACTIVE_PROJECT[] = nothing

src/Pkg.jl

@@ -215,7 +215,7 @@ by starting julia with `--inline=no`.
 const test = API.test
 
 """
-    Pkg.gc()
+    Pkg.gc(; io::IO=DEFAULT_IO[])
 
 Garbage collect packages that are no longer reachable from any project.
 Only packages that are tracked by version are deleted, so no packages
@@ -225,9 +225,9 @@ const gc = API.gc
 
 
 """
-    Pkg.build(; verbose = false)
-    Pkg.build(pkg::Union{String, Vector{String}}; verbose = false)
-    Pkg.build(pkgs::Union{PackageSpec, Vector{PackageSpec}}; verbose = false)
+    Pkg.build(; verbose = false, io::IO=DEFAULT_IO[])
+    Pkg.build(pkg::Union{String, Vector{String}}; verbose = false, io::IO=DEFAULT_IO[])
+    Pkg.build(pkgs::Union{PackageSpec, Vector{PackageSpec}}; verbose = false, io::IO=DEFAULT_IO[])
 
 Run the build script in `deps/build.jl` for `pkg` and all of its dependencies in
 depth-first recursive order.
@@ -241,8 +241,8 @@ redirecting to the `build.log` file.
 const build = API.build
 
 """
-    Pkg.pin(pkg::Union{String, Vector{String}})
-    Pkg.pin(pkgs::Union{PackageSpec, Vector{PackageSpec}})
+    Pkg.pin(pkg::Union{String, Vector{String}}; io::IO=DEFAULT_IO[])
+    Pkg.pin(pkgs::Union{PackageSpec, Vector{PackageSpec}}; io::IO=DEFAULT_IO[])
 
 Pin a package to the current version (or the one given in the `PackageSpec`) or to a certain
 git revision. A pinned package is never updated.
@@ -256,8 +256,8 @@ Pkg.pin(name="Example", version="0.3.1")
 const pin = API.pin
 
 """
-    Pkg.free(pkg::Union{String, Vector{String}})
-    Pkg.free(pkgs::Union{PackageSpec, Vector{PackageSpec}})
+    Pkg.free(pkg::Union{String, Vector{String}}; io::IO=DEFAULT_IO[])
+    Pkg.free(pkgs::Union{PackageSpec, Vector{PackageSpec}}; io::IO=DEFAULT_IO[])
 
 If `pkg` is pinned, remove the pin.
 If `pkg` is tracking a path,
@@ -272,8 +272,8 @@ const free = API.free
 
 
 """
-    Pkg.develop(pkg::Union{String, Vector{String}})
-    Pkg.develop(pkgs::Union{Packagespec, Vector{Packagespec}})
+    Pkg.develop(pkg::Union{String, Vector{String}}; io::IO=DEFAULT_IO[])
+    Pkg.develop(pkgs::Union{Packagespec, Vector{Packagespec}}; io::IO=DEFAULT_IO[])
 
 Make a package available for development by tracking it by path.
 If `pkg` is given with only a name or by a URL, the package will be downloaded
@@ -346,7 +346,7 @@ Request a `ProjectInfo` struct which contains information about the active proje
 const project = API.project
 
 """
-    Pkg.instantiate(; verbose = false)
+    Pkg.instantiate(; verbose = false, io::IO=DEFAULT_IO[])
 
 If a `Manifest.toml` file exists in the active project, download all
 the packages declared in that manifest.
@@ -360,15 +360,15 @@ dependencies in the manifest and instantiate the resulting project.
 const instantiate = API.instantiate
 
 """
-    Pkg.resolve()
+    Pkg.resolve(; io::IO=DEFAULT_IO[])
 
 Update the current manifest with potential changes to the dependency graph
 from packages that are tracking a path.
 """
 const resolve = API.resolve
 
 """
-    Pkg.status([pkgs...]; mode::PackageMode=PKGMODE_PROJECT, diff::Bool=false)
+    Pkg.status([pkgs...]; mode::PackageMode=PKGMODE_PROJECT, diff::Bool=false, io::IO=stdout)
 
 Print out the status of the project/manifest.
 If `mode` is `PKGMODE_PROJECT`, print out status only about the packages
@@ -389,7 +389,7 @@ const status = API.status
 
 
 """
-    Pkg.activate([s::String]; shared::Bool=false)
+    Pkg.activate([s::String]; shared::Bool=false, io::IO=DEFAULT_IO[])
 
 Activate the environment at `s`. The active environment is the environment
 that is modified by executing package commands.

test/new.jl

@@ -2521,7 +2521,10 @@ end
 @testset "STDLIBS_BY_VERSION up-to-date" begin
     test_result = Pkg.Types.STDLIBS_BY_VERSION[end][2] == Pkg.Types.load_stdlib()
     if !test_result
-        @error("STDLIBS_BY_VERSION out of date!  Re-run generate_historical_stdlibs.jl!")
+        @error("STDLIBS_BY_VERSION out of date!  Manually fix given the info below, or re-run generate_historical_stdlibs.jl!")
+        @show length(Pkg.Types.STDLIBS_BY_VERSION[end][2]) length(Pkg.Types.load_stdlib())
+        @show setdiff(Pkg.Types.STDLIBS_BY_VERSION[end][2], Pkg.Types.load_stdlib())
+        @show setdiff(Pkg.Types.load_stdlib(), Pkg.Types.STDLIBS_BY_VERSION[end][2])
     end
     @test test_result
 end