Skip to content
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

0.6.0 #7

Merged
merged 57 commits into from
Aug 15, 2023
Merged

0.6.0 #7

merged 57 commits into from
Aug 15, 2023

Conversation

acheronfail
Copy link
Owner

@acheronfail acheronfail commented Jul 6, 2023

Notable Changes

  • Overhaul of network detection - it now uses netlink directly, and removes the dependency on NetworkManager as well as iwlib! 🎉
  • reset bar item retries after a timeout (only stop restarting them if they restart many times in a short period)
  • istat now exits with 0 when the shutdown ipc event is sent

Other changes

  • dependency updates
  • create Result and Error types to avoid having to use Box<dyn Error> everywhere
  • fixes to local dev loops

Work Remaining

@acheronfail acheronfail force-pushed the next branch 6 times, most recently from fae1e63 to fae1202 Compare July 14, 2023 00:40
@acheronfail acheronfail force-pushed the next branch 3 times, most recently from fae1fb2 to fae18f2 Compare July 27, 2023 01:35
@acheronfail acheronfail force-pushed the next branch 6 times, most recently from fae192b to fae188d Compare August 6, 2023 10:25
And only updating a single interface at a time doesn't seem worth it.
I've tried it a few times, and keep coming up short.

This link could be somewhat useful in the future though:
http://www.infradead.org/~tgr/libnl/doc/route.html
A couple issues:

- `krb` wasn't working in the tests because there was a filter in the
  sample_config.toml file
- `krb` if there were no filters, then the first update would occur only
  _after_ the first interval
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant