Skip to content

Commit

Permalink
Ensure --depwarn=yes by default in Pkg.test (#1763)
Browse files Browse the repository at this point in the history
* Ensure --depwarn=yes by default in Pkg.test

* Respect --depwarn=error in parent process
  • Loading branch information
c42f authored Apr 10, 2020
1 parent f6ac201 commit 2874057
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 0 deletions.
1 change: 1 addition & 0 deletions src/Operations.jl
Original file line number Diff line number Diff line change
Expand Up @@ -1282,6 +1282,7 @@ function gen_test_code(testfile::String;
--color=$(Base.have_color === nothing ? "auto" : Base.have_color ? "yes" : "no")
--compiled-modules=$(Bool(Base.JLOptions().use_compiled_modules) ? "yes" : "no")
--check-bounds=yes
--depwarn=$(Base.JLOptions().depwarn == 2 ? "error" : "yes")
--inline=$(Bool(Base.JLOptions().can_inline) ? "yes" : "no")
--startup-file=$(Base.JLOptions().startupfile == 1 ? "yes" : "no")
--track-allocation=$(("none", "user", "all")[Base.JLOptions().malloc_log + 1])
Expand Down
1 change: 1 addition & 0 deletions test/test_packages/TestArguments/test/runtests.jl
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
@assert ARGS == ["a", "b"]
@assert Base.JLOptions().quiet == 1 # --quiet
@assert Base.JLOptions().check_bounds == 2 # overriden default when testing
@assert Base.JLOptions().depwarn == 1 # Default to depwarn on

0 comments on commit 2874057

Please sign in to comment.