Add ring_logger by default

Author: mobileoverlord

lib/mix/tasks/nerves/new.ex

@@ -8,6 +8,7 @@ defmodule Mix.Tasks.Nerves.New do
   @shoehorn_vsn "0.4"
   @bootstrap_vsn "1.0"
   @runtime_vsn "0.6"
+  @ring_logger_vsn "0.4"
   @init_gadget_vsn "0.4"
 
   @requirement Mix.Project.config()[:elixir]
@@ -167,6 +168,7 @@ defmodule Mix.Tasks.Nerves.New do
       bootstrap_vsn: @bootstrap_vsn,
       shoehorn_vsn: @shoehorn_vsn,
       runtime_vsn: @runtime_vsn,
+      ring_logger_vsn: @ring_logger_vsn,
       elixir_req: @requirement,
       nerves_dep: nerves_dep(nerves_path),
       in_umbrella: in_umbrella?,

templates/new/config/config.exs

@@ -14,7 +14,13 @@ config :nerves, :firmware, rootfs_overlay: "rootfs_overlay"
 # involved with firmware updates.
 config :shoehorn,
   init: [:nerves_runtime<%= if init_gadget? do %>, :nerves_init_gadget<% end %>],
-  app: Mix.Project.config()[:app]<%= if init_gadget? do %>
+  app: Mix.Project.config()[:app]
+  
+# Use Ringlogger as the logger backend and remove :console.
+# See https://hexdocs.pm/ring_logger/readme.html for more information on
+# configuring ring_logger.
+
+config :logger, backends: [RingLogger]<%= if init_gadget? do %>
 
 # Authorize the device to receive firmware using your public key.
 # See https://hexdocs.pm/nerves_firmware_ssh/readme.html for more information
@@ -29,12 +35,6 @@ config :nerves_firmware_ssh,
     File.read!(key)
   ]
 
-# Use Ringlogger as the logger backend and remove :console.
-# See https://hexdocs.pm/ring_logger/readme.html for more information on
-# configuring ring_logger.
-
-config :logger, backends: [RingLogger]
-
 # Configure nerves_init_gadget.
 # See https://hexdocs.pm/nerves_init_gadget/readme.html for more information.
 

templates/new/mix.exs

@@ -42,7 +42,8 @@ defmodule <%= app_module %>.MixProject do
   defp deps do
     [
       <%= nerves_dep %>,
-      {:shoehorn, "~> <%= shoehorn_vsn %>"}
+      {:shoehorn, "~> <%= shoehorn_vsn %>"},
+      {:ring_logger, "~> <%= ring_logger_vsn %>"}
     ] ++ deps(@target)
   end
 

test/nerves_new_test.exs

@@ -101,6 +101,20 @@ defmodule Nerves.NewTest do
     end)
   end
 
+  test "new project includes ring_logger", context do
+    in_tmp(context.test, fn ->
+      Mix.Tasks.Nerves.New.run([@app_name])
+
+      assert_file("#{@app_name}/mix.exs", fn file ->
+        assert file =~ ~r":ring_logger"
+      end)
+
+      assert_file("#{@app_name}/config/config.exs", fn file ->
+        assert file =~ ~r"RingLogger"
+      end)
+    end)
+  end
+
   test "new project init gadget", context do
     in_tmp(context.test, fn ->
       Mix.Tasks.Nerves.New.run([@app_name, "--init-gadget"])
@@ -112,7 +126,6 @@ defmodule Nerves.NewTest do
       assert_file("#{@app_name}/config/config.exs", fn file ->
         assert file =~ ~r"nerves_init_gadget"
         assert file =~ ~r"nerves_firmware_ssh"
-        assert file =~ ~r"RingLogger"
       end)
     end)
   end