Update to embassy-net 0.5

[ivmarkov]

(Not immediately as we might have to wait until esp-wifi gets it as well.)

[MabezDev]

Not immediately as we might have to wait until esp-wifi gets it as well.

What do you need on the esp-wifi side? embassy-net-driver didn't change so you should be able to use embassy-net@0.5 with the currently released esp-wifi... in theory :D, let me know if that's not the case.

[ivmarkov]

Not immediately as we might have to wait until esp-wifi gets it as well.

What do you need on the esp-wifi side? embassy-net-driver didn't change so you should be able to use embassy-net@0.5 with the currently released esp-wifi... in theory :D, let me know if that's not the case.

Ah right - you are only depending on the -driver thing. Even better then!

[lu-zero]

Current esp-wifi uses smoltcp-0.11, from the user-side you want to have everything on the same smoltcp version.

[MabezDev]

True, but it doesn't stop you using embassy-net because it uses its own device trait. I opened an upstream issue smoltcp to do the same btw: smoltcp-rs/smoltcp#1010

[lu-zero]

Yes, it is just mildly annoying since you have to put in the configuration for smoltcp twice like I did in wot-rust/wot-esp-hal-demo#14

[lu-zero]

Had been a while, do you have time to land it and cut a release? Doesn't look like we'll have an esp-wifi update much soon ^^;

[ivmarkov]

Had been a while, do you have time to land it and cut a release? Doesn't look like we'll have an esp-wifi update much soon ^^;

Is it working fine for you in the meantime?
Don't want to cut a release which does not work...

[lu-zero]

I had a branch for the mdns support and it works well. There is the pitfall of having to configure two smoltcp but everything works.

[ivmarkov]

I had a branch for the mdns support and it works well. There is the pitfall of having to configure two smoltcp but everything works.

Alright. Another user in the Embassy matrix chat was trying it out with a pi pico w I think and was having troubles getting the incoming multicast traffic (outgoing was working fine) but I doubt this can be a problem of edge-net if you say it works for you with baremetal esp.

I'll merge in a few days and release.

[lu-zero]

I can try to test it more in the next days if you prefer, today I'm far a bit far from the board.

[AnthonyGrondin]

Looking forward to see this being merged, so that I can upgrade to embedded-nal-async@0.8.0 across all my packages. embassy-net@0.4.0 still depends on embedded-nal-async@0.7.1.

I've been testing a bit in the following, using a patched version for this PR:

• Update smoltcp to 0.12.0 and embassy-net to 0.5.0 esp-rs/esp-hal#2849

[lu-zero]

If that lands we do not have anymore the pitfall of having two smoltcp in the deptree :)