-
Notifications
You must be signed in to change notification settings - Fork 16
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
zbus is a heavy dependency #15
Comments
If you would like to open a PR replacing it, then I'd review it |
Please do not replace zbus with dbus! dbus requires a C library. zbus is pure Rust. Those statistics are misleading. They do not account for the libdbus C library and its dependencies. |
Maybe we could have it behind a feature flag? So you can choose one or the other |
If you insist, but IMO it'd be pretty silly to have two implementations of the same thing. |
Nah I don't insist, I'd rather have a pure-Rust dependency tree |
For reference: https://gitlab.freedesktop.org/dbus/zbus/-/blob/main/zbus/Cargo.toml |
Many zbus dependencies are optional, as per the latest release: https://gitlab.freedesktop.org/dbus/zbus/-/releases So zbus might be lighter now |
I want to push back a bit on this one, because I anticipate the same complaint about AccessKit's upcoming zbus-based AT-SPI implementation, and because I think choosing light versus dark mode is an accessibility issue in itself. While we should certainly avoid adding gratuitous complexity and code bloat, we should also keep in mind that a human interface that accommodates the needs of a variety of users is fundamentally not a simple, small problem, so it can't have a simple, small solution. Whether or not D-Bus is over-complicated, it's what we have on the free desktops, and meeting the needs of users must come first. I think dark-light is right to prefer a pure-Rust dependency tree. So @frewsxcv, you should feel free to dismiss this issue. |
Is zbus any lighter today than when this issue was made? @frewsxcv |
I have no idea |
I think zbus is actually smaller now? |
https://lib.rs/crates/zbus gives a rough estimate of weight. When I created this, its dependencies were estimated to be 5-7MB. It's now 10-26MB. |
Hmm, there's not really any alternative in the rust world though is there? I suppose dark-light could make z/dbus a feature that can be installed separately |
No, there isn't a Rust alternative to zbus. There's no actionable request here. If you don't like zbus, improve it or write an alternative. |
It is actionable, which is different from maintainers having no interest. |
There is no actionable request. |
zbus
replaceddbus
in edff243 and the change was requested in #4 (comment)Some stats according to
lib.rs
:Related: https://gitlab.freedesktop.org/dbus/zbus/-/issues/254
The text was updated successfully, but these errors were encountered: