-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
uf2tool is included in device avm #44
Comments
Good catch! It did not even cross my mind that could be a possibility when I wrote the task. We should double check that the pico flash, and stm32 flash tasks are not in there too, since I also contributed those. ;-) |
uf2tool is there because it's a dep. Believe "Mix is not part of the runtime env", which is also why we have that deprecation/migration Mix.Config->Config afaik, so they should not be there.. But we should of course check ;-) |
Another reason we might actually want to keep the current Mix.Config... it's guaranteed not to leek into the runtime environment. |
these all currently works: ExAtomVM.hello()
|> IO.inspect()
ExAtomVM.PackBEAM.module_info()
|> IO.inspect()
Mix.Tasks.Atomvm.Check.module_info()
|> IO.inspect() My current thought is install instructions should include
that correctly gives you:
|
so we need to add at least
runtime: false
, if notonly: :dev
as well in mix.exs..exatomvm/lib/mix/tasks/packbeam.ex
Line 163 in cbd2fb6
The text was updated successfully, but these errors were encountered: