Add :autoupdate option to app config and use one top-level supervisor.

README.md

@@ -51,6 +51,12 @@ For use with [Calendar](https://github.com/lau/calendar) you can still
 specify tzdata ~> 0.1.7 in your mix.exs file in case you experience problems
 using version ~> 0.5.2.
 
+You can also disable the automatic downloading of new timezone databases:
+
+```elixir
+config :tzdata, :autoupdate, :disabled
+```
+
 ## Documentation
 
 Documentation can be found at http://hexdocs.pm/tzdata/

config/config.exs

@@ -2,23 +2,4 @@
 # and its dependencies with the aid of the Mix.Config module.
 use Mix.Config
 
-# This configuration is loaded before any dependency and is restricted
-# to this project. If another project depends on this project, this
-# file won't be loaded nor affect the parent project. For this reason,
-# if you want to provide default values for your application for third-
-# party users, it should be done in your mix.exs file.
-
-# Sample configuration:
-#
-#     config :logger, :console,
-#       level: :info,
-#       format: "$date $time [$level] $metadata$message\n",
-#       metadata: [:user_id]
-
-# It is also possible to import configuration files, relative to this
-# directory. For example, you can emulate configuration per environment
-# by uncommenting the line below and defining dev.exs, test.exs and such.
-# Configuration from the imported file will override the ones defined
-# here (which is why it is important to import them last).
-#
-#     import_config "#{Mix.env}.exs"
+config :tzdata, :autoupdate, :enabled

lib/tzdata/ets_holder.ex

@@ -2,11 +2,11 @@ defmodule Tzdata.EtsHolder do
   require Logger
   use GenServer
   alias Tzdata.DataBuilder
-  def start_link(_) do
+  def start_link() do
     GenServer.start_link(__MODULE__, [], name: __MODULE__)
   end
 
-  def init(_) do
+  def init([]) do
     make_sure_a_release_is_on_file
     create_current_release_ets_table
     {:ok, release_name} = load_release

lib/tzdata/release_updater.ex

@@ -3,11 +3,11 @@ defmodule Tzdata.ReleaseUpdater do
   use GenServer
   alias Tzdata.DataLoader
 
-  def start_link(_) do
+  def start_link() do
     GenServer.start_link(__MODULE__, [], name: :tzdata_release_updater)
   end
 
-  def init(_) do
+  def init([]) do
     Task.async(fn -> :timer.sleep(3000); Tzdata.ReleaseUpdater.check_if_time_to_update end)
     {:ok, []}
   end

lib/tzdata/tzdata_app.ex

@@ -2,7 +2,16 @@ defmodule Tzdata.App do
   use Application
 
   def start(_type, _args) do
-    Tzdata.EtsHolderSupervisor.start_link
-    Tzdata.ReleaseUpdaterSupervisor.start_link
+    import Supervisor.Spec
+
+    children = [
+      worker(Tzdata.EtsHolder, [])
+    ]
+    children = case Application.fetch_env(:tzdata, :autoupdate) do
+      {:ok, :enabled} -> children ++ [worker(Tzdata.ReleaseUpdater, [])]
+      {:ok, :disabled} -> children
+    end
+
+    {:ok, _} = Supervisor.start_link(children, strategy: :one_for_one)
   end
 end

mix.exs

@@ -12,8 +12,10 @@ defmodule Tzdata.Mixfile do
   end
 
   def application do
-    [applications: [:hackney, :logger],
-    mod: {Tzdata.App, []}
+    [
+      applications: [:hackney, :logger],
+      env: env,
+      mod: {Tzdata.App, []}
     ]
   end
 
@@ -25,6 +27,10 @@ defmodule Tzdata.Mixfile do
     ]
   end
 
+  defp env do
+    [autoupdate: true]
+  end
+
   defp description do
     """
     Tzdata is a parser and library for the tz database.