update dependencies

[jxs]

Linux (linux.rs):

The netlink-packet-route crate renamed nlas::Nla (NetLink Attributes) to AddressAttribute, and its new API directly contains std::net::IpAddr instead of raw bytes, eliminating the need to manually parse byte arrays and check the address family.

Apple (apple.rs):

The system-configuration crate changed SCDynamicStoreBuilder::build() and SCDynamicStore::create_run_loop_source() to return Option<T> instead of T directly.

Windows (win.rs):

The windows crate changed NotifyIpInterfaceChange() and CancelMibChangeNotify2() to return WIN32_ERROR directly instead of Result<()>, requiring a call to .ok() to convert to a Result. Additionally, the BOOLEAN wrapper type was replaced with native bool.

I plan to release a patch release after this gets merged

[elenaf9]

Shouldn't we return the error here? IfWatcher::new (which calls background_task) already returns a Result, so we could propagate it to the user.

[jxs]

yeah but it calls background_task inside a spawned thread not in the context of the new function itself

[elenaf9]

True. But we could split it into, e.g., create_task and run_task or sth like that. I would prefer if we don't just silently fail (apart from the log msg).

[jxs]

ah right, yeah that would make sense, I tried it but CFRunLoopSource is !Send

[elenaf9]

Discussed out of band. New solution of handling the a failing background_task by polling rx in poll_if_event lgtm!

[elenaf9]

LGTM, thanks!

[elenaf9]

Discussed out of band. New solution of handling the a failing background_task by polling rx in poll_if_event lgtm!

[nazar-pc]

Thanks for updating dependencies, is there a new release planned here?
Patch version would be nice since API wasn't affected.

[elenaf9]

Thanks for updating dependencies, is there a new release planned here? Patch version would be nice since API wasn't affected.

Released v3.2.2: #53 (comment).