Skip to content

Commit

Permalink
Change include_transitive_dependencies default to false
Browse files Browse the repository at this point in the history
This seems like the easiest fix to #639.
  • Loading branch information
simonbyrne authored Jan 2, 2022
1 parent 22576ee commit 998a514
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions src/PackageCompiler.jl
Original file line number Diff line number Diff line change
Expand Up @@ -389,7 +389,7 @@ compiler (can also include extra arguments to the compiler, like `-g`).
- `include_transitive_dependencies::Bool`: If `true`, explicitly put all
transitive dependencies into the sysimage. This only makes a difference if some
packages do not load all their dependencies when themselves are loaded. Defaults to `true`.
packages do not load all their dependencies when themselves are loaded. Defaults to `false`.
### Advanced keyword arguments
Expand All @@ -415,7 +415,7 @@ function create_sysimage(packages::Union{Nothing, Symbol, Vector{String}, Vector
cpu_target::String=NATIVE_CPU_TARGET,
script::Union{Nothing, String}=nothing,
sysimage_build_args::Cmd=``,
include_transitive_dependencies::Bool=true,
include_transitive_dependencies::Bool=false,
# Internal args
base_sysimage::Union{Nothing, String}=nothing,
julia_init_c_file=nothing,
Expand Down Expand Up @@ -664,7 +664,7 @@ compiler (can also include extra arguments to the compiler, like `-g`).
- `include_transitive_dependencies::Bool`: If `true`, explicitly put all
transitive dependencies into the sysimage. This only makes a difference if some
packages do not load all their dependencies when themselves are loaded. Defaults to `true`.
packages do not load all their dependencies when themselves are loaded. Defaults to `false`.
### Advanced keyword arguments
Expand All @@ -685,7 +685,7 @@ function create_app(package_dir::String,
cpu_target::String=default_app_cpu_target(),
include_lazy_artifacts::Bool=false,
sysimage_build_args::Cmd=``,
include_transitive_dependencies::Bool=true)
include_transitive_dependencies::Bool=false)
warn_official()

ctx = create_pkg_context(package_dir)
Expand Down Expand Up @@ -834,7 +834,7 @@ compiler (can also include extra arguments to the compiler, like `-g`).
- `include_transitive_dependencies::Bool`: If `true`, explicitly put all
transitive dependencies into the sysimage. This only makes a difference if some
packages do not load all their dependencies when themselves are loaded. Defaults to `true`.
packages do not load all their dependencies when themselves are loaded. Defaults to `false`.
### Advanced keyword arguments
Expand All @@ -858,7 +858,7 @@ function create_library(package_dir::String,
cpu_target::String=default_app_cpu_target(),
include_lazy_artifacts::Bool=false,
sysimage_build_args::Cmd=``,
include_transitive_dependencies::Bool=true)
include_transitive_dependencies::Bool=false)


warn_official()
Expand Down

0 comments on commit 998a514

Please sign in to comment.