Merge pull request #102 from chengchingwen/master

make env variable name consistent

Project.toml

@@ -1,7 +1,7 @@
 name = "DataDeps"
 uuid = "124859b0-ceae-595e-8997-d05f6a7a8dfe"
 authors = ["Lyndon White <oxinabox@ucc.asn.au>"]
-version = "0.6.4"
+version = "0.6.5"
 
 [deps]
 HTTP = "cd3eb016-35fb-5094-929b-558a96fad6f3"
@@ -10,7 +10,7 @@ SHA = "ea8e919c-243c-51af-8825-aaa63cd721ce"
 
 [compat]
 ExpectationStubs = "0.2.0"
-HTTP = "0.7.0, 0.8"
+HTTP = "0.8.5"
 julia = "0.7, 1"
 
 [extras]

docs/src/z10-for-end-users.md

@@ -108,7 +108,7 @@ DataDeps.jl tries to have very sensible defaults.
     - Note that it remains your responsibility to understand and read any terms of the data use (this is remains true even if you don't turn on this bypass)    
     - This is provided for scripting (in particular CI) use
     - If the `CI` environment variable is set to true, `DATADEPS_ALWAYS_ACCEPT`  **must be set** to true or false. This is to prevent hanging in CI.
- - `DATADEP_PROGRESS_UPDATE_PERIOD` -- how often (in seconds) to print the progress to the log for the download
+ - `DATADEPS_PROGRESS_UPDATE_PERIOD` -- how often (in seconds) to print the progress to the log for the download
 	- This is used by the default `fetch_method` and when implementing custom methods it is good to respect it.
 	- default: `5` (seconds) usually; `Inf` (i.e. no updates) if `DATADEPS_ALWAYS_ACCEPT` is set.
  - `DATADEPS_LOAD_PATH` -- The list of paths to be prepended to the standard loadpath to save and load data from

src/fetch_helpers.jl

@@ -9,12 +9,17 @@ This is used by the default `fetch_method` and is generally a good idea
 to use it in any custom fetch method, if possible
 """
 function progress_update_period()
-    envvar = get(ENV, "DATADEP_PROGRESS_UPDATE_PERIOD") do
-        if haskey(ENV, "DATADEP_ALWAYS_ACCEPT")
+    envvar = get(ENV, "DATADEPS_PROGRESS_UPDATE_PERIOD") do
+        if haskey(ENV, "DATADEPS_ALWAYS_ACCEPT")
             # Running in a script, probably want minimal updates
             "Inf"
         else # default
-            "5" # seconds
+            if haskey(ENV, "DATADEP_PROGRESS_UPDATE_PERIOD")
+                @warn "\"DATADEP_PROGRESS_UPDATE_PERIOD\" is deprecated, use \"DATADEPS_PROGRESS_UPDATE_PERIOD\" instead."
+                get(ENV, "DATADEP_PROGRESS_UPDATE_PERIOD", "5")
+            else
+                "5" # seconds
+            end
         end
     end
     parse(Float32, envvar)