Skip to content

Remove futures-lite#62

Merged
Be-ing merged 3 commits intomainfrom
remove/futures-lite
Jan 9, 2025
Merged

Remove futures-lite#62
Be-ing merged 3 commits intomainfrom
remove/futures-lite

Conversation

@edfloreshz
Copy link
Collaborator

@edfloreshz edfloreshz commented Jan 9, 2025

This pull request removes futures-lite in favor of async-std as we are already using it with ashpd, fixes issues with the Error enum and removes unnecessary dependencies.

Improvements:

  • src/platforms/freedesktop.rs: Implemented timeout handling using async-std in the detect function to avoid latency issues.
  • src/error.rs: Updated Error enum and related implementations to support FreeBSD, DragonFly, NetBSD, and OpenBSD in addition to Linux.
  • src/error.rs: Added a new Timeout variant to the Error enum to handle timeout errors.
  • Cargo.toml: Updated dependencies by removing futures-lite and tokio, and adding async-std.

Fixes #17

@edfloreshz edfloreshz added the enhancement New feature or request label Jan 9, 2025
@edfloreshz edfloreshz requested a review from Be-ing January 9, 2025 19:48
@edfloreshz edfloreshz self-assigned this Jan 9, 2025
@edfloreshz edfloreshz force-pushed the remove/futures-lite branch 2 times, most recently from 9d9c85e to 316c788 Compare January 9, 2025 20:34
@edfloreshz edfloreshz force-pushed the remove/futures-lite branch from 316c788 to 925979f Compare January 9, 2025 20:34
@Be-ing Be-ing merged commit b10982b into main Jan 9, 2025
4 checks passed
@edfloreshz edfloreshz mentioned this pull request Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Latency issues on Linux

2 participants