Fix: Packbeam support runtime: false & avoid duplicates

Signed-off-by: Peter M <[email protected]>

Author: petermm

lib/mix/tasks/packbeam.ex

@@ -158,10 +158,12 @@ defmodule Mix.Tasks.Atomvm.Packbeam do
     end
   end
 
-  def runtime_deps(deps) do
+  def runtime_deps(deps, is_runtime_dep \\ false) do
     Enum.reduce(deps, [], fn dep, acc ->
-      if Keyword.get(dep.opts, :runtime, true) do
-        ["#{dep.opts[:build]}/ebin" | runtime_deps(dep.deps) ++ acc]
+      if Keyword.get(dep.opts, :runtime, true) and
+           (is_runtime_dep == true or dep.top_level == true) and
+           dep.app != :exatomvm do
+        ["#{dep.opts[:build]}/ebin" | runtime_deps(dep.deps, true) ++ acc]
       else
         acc
       end

lib/packbeam.ex

@@ -130,7 +130,9 @@ defmodule ExAtomVM.PackBEAM do
   end
 
   defp pack_files(modules) do
-    Enum.reduce_while(modules, {:ok, []}, fn {module, opts}, {:ok, acc} ->
+    modules
+    |> Enum.uniq()
+    |> Enum.reduce_while({:ok, []}, fn {module, opts}, {:ok, acc} ->
       case pack_file(module, opts) do
         {:ok, res} -> {:cont, {:ok, [acc | res]}}
         error -> {:halt, error}