nerves icon indicating copy to clipboard operation
nerves copied to clipboard

Add mix discovery command for device discovery

Open lawik opened this issue 1 year ago • 2 comments

Uses :mndp library and delegates to mix mndp.discover. If we switch discovery method over time this means we can fairly easily switch without changing the command.

It requires MNDP to be running in your app and a matching change to mix nerves.new might be reasonable to include mndp.

lawik avatar Oct 21 '24 08:10 lawik

I am not sure we are happy to put this in yet or if mndp needs some time to bake. I really like what it would bring in terms of dev convenience.

You run mix discover and get:

Screenshot_20241021_103654

This doesn't have the risk of collision with system implementations and general weirdness that we get with mDNS. Maybe we will find other weirdness? I don't know. But it seems rad.

lawik avatar Oct 21 '24 08:10 lawik

nerves_bootstrap PR: https://github.com/nerves-project/nerves_bootstrap/pull/332

lawik avatar Oct 21 '24 08:10 lawik

@lawik I really like MNDP, but I think I need to spend some time using this in anger before elevating it to Nerves core tooling. The good part is that everyone can use mix mndp.discover right now without any changes to Nerves.

fhunleth avatar Oct 21 '24 12:10 fhunleth

Entirely fair. I'm fine with these PRs idling or being closed and we go in for it when it feels ready.

lawik avatar Oct 21 '24 12:10 lawik

We're moving to use igniter for installing everything, so closing in anticipation for working on that.

fhunleth avatar May 14 '25 08:05 fhunleth