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.

Sign up for GitHub

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

Jump to bottom

Reduce or Feature gate dependencies #140

Closed
coderedart opened this issue Aug 30, 2023 · 7 comments
Closed

Reduce or Feature gate dependencies #140

coderedart opened this issue Aug 30, 2023 · 7 comments

Comments

@coderedart
Copy link

coderedart commented Aug 30, 2023 •
edited
Loading

I would like to preface this by saying that i am ignorant in this area. So, i apologize in advance if i am asking a dumb question.

But is it possible to put all those async dependencies behind feature flags for async dialogs? or do we need those for the blocking operations too (especially on windows/linux)?

with all features disabled, this is the output of cargo tree 😢 (over 50 deps, mostly from async-io)

└── rfd v0.11.4
    ├── async-io v1.13.0
    │   ├── async-lock v2.8.0
    │   │   └── event-listener v2.5.3
    │   ├── cfg-if v1.0.0
    │   ├── concurrent-queue v2.2.0
    │   │   └── crossbeam-utils v0.8.16
    │   │       └── cfg-if v1.0.0
    │   ├── futures-lite v1.13.0
    │   │   ├── fastrand v1.9.0
    │   │   ├── futures-core v0.3.28
    │   │   ├── futures-io v0.3.28
    │   │   ├── memchr v2.6.1
    │   │   ├── parking v2.1.0
    │   │   ├── pin-project-lite v0.2.13
    │   │   └── waker-fn v1.1.0
    │   ├── log v0.4.20
    │   ├── parking v2.1.0
    │   ├── polling v2.8.0
    │   │   ├── cfg-if v1.0.0
    │   │   ├── libc v0.2.147
    │   │   └── log v0.4.20
    │   │   [build-dependencies]
    │   │   └── autocfg v1.1.0
    │   ├── rustix v0.37.23
    │   │   ├── bitflags v1.3.2
    │   │   ├── io-lifetimes v1.0.11
    │   │   │   └── libc v0.2.147
    │   │   └── linux-raw-sys v0.3.8
    │   ├── slab v0.4.9
    │   │   [build-dependencies]
    │   │   └── autocfg v1.1.0
    │   ├── socket2 v0.4.9
    │   │   └── libc v0.2.147
    │   └── waker-fn v1.1.0
    │   [build-dependencies]
    │   └── autocfg v1.1.0
    ├── futures-util v0.3.28
    │   ├── futures-core v0.3.28
    │   ├── futures-io v0.3.28
    │   ├── futures-task v0.3.28
    │   ├── memchr v2.6.1
    │   ├── pin-project-lite v0.2.13
    │   ├── pin-utils v0.1.0
    │   └── slab v0.4.9 (*)
    ├── log v0.4.20
    └── raw-window-handle v0.5.2

And here i have xdg portal feature enabled for linux 😭 😭 😭 (over 150 dependencies, mostly from ashpd. )

rfd v0.11.4
    ├── ashpd v0.4.0
    │   ├── async-std v1.12.0
    │   │   ├── async-channel v1.9.0
    │   │   │   ├── concurrent-queue v2.2.0
    │   │   │   │   └── crossbeam-utils v0.8.16
    │   │   │   │       └── cfg-if v1.0.0
    │   │   │   ├── event-listener v2.5.3
    │   │   │   └── futures-core v0.3.28
    │   │   ├── async-global-executor v2.3.1
    │   │   │   ├── async-channel v1.9.0 (*)
    │   │   │   ├── async-executor v1.5.1
    │   │   │   │   ├── async-lock v2.8.0
    │   │   │   │   │   └── event-listener v2.5.3
    │   │   │   │   ├── async-task v4.4.0
    │   │   │   │   ├── concurrent-queue v2.2.0 (*)
    │   │   │   │   ├── fastrand v1.9.0
    │   │   │   │   ├── futures-lite v1.13.0
    │   │   │   │   │   ├── fastrand v1.9.0
    │   │   │   │   │   ├── futures-core v0.3.28
    │   │   │   │   │   ├── futures-io v0.3.28
    │   │   │   │   │   ├── memchr v2.6.1
    │   │   │   │   │   ├── parking v2.1.0
    │   │   │   │   │   ├── pin-project-lite v0.2.13
    │   │   │   │   │   └── waker-fn v1.1.0
    │   │   │   │   └── slab v0.4.9
    │   │   │   │       [build-dependencies]
    │   │   │   │       └── autocfg v1.1.0
    │   │   │   ├── async-io v1.13.0
    │   │   │   │   ├── async-lock v2.8.0 (*)
    │   │   │   │   ├── cfg-if v1.0.0
    │   │   │   │   ├── concurrent-queue v2.2.0 (*)
    │   │   │   │   ├── futures-lite v1.13.0 (*)
    │   │   │   │   ├── log v0.4.20
    │   │   │   │   │   └── value-bag v1.4.1
    │   │   │   │   ├── parking v2.1.0
    │   │   │   │   ├── polling v2.8.0
    │   │   │   │   │   ├── cfg-if v1.0.0
    │   │   │   │   │   ├── libc v0.2.147
    │   │   │   │   │   └── log v0.4.20 (*)
    │   │   │   │   │   [build-dependencies]
    │   │   │   │   │   └── autocfg v1.1.0
    │   │   │   │   ├── rustix v0.37.23
    │   │   │   │   │   ├── bitflags v1.3.2
    │   │   │   │   │   ├── io-lifetimes v1.0.11
    │   │   │   │   │   │   └── libc v0.2.147
    │   │   │   │   │   └── linux-raw-sys v0.3.8
    │   │   │   │   ├── slab v0.4.9 (*)
    │   │   │   │   ├── socket2 v0.4.9
    │   │   │   │   │   └── libc v0.2.147
    │   │   │   │   └── waker-fn v1.1.0
    │   │   │   │   [build-dependencies]
    │   │   │   │   └── autocfg v1.1.0
    │   │   │   ├── async-lock v2.8.0 (*)
    │   │   │   ├── blocking v1.3.1
    │   │   │   │   ├── async-channel v1.9.0 (*)
    │   │   │   │   ├── async-lock v2.8.0 (*)
    │   │   │   │   ├── async-task v4.4.0
    │   │   │   │   ├── atomic-waker v1.1.1
    │   │   │   │   ├── fastrand v1.9.0
    │   │   │   │   ├── futures-lite v1.13.0 (*)
    │   │   │   │   └── log v0.4.20 (*)
    │   │   │   ├── futures-lite v1.13.0 (*)
    │   │   │   └── once_cell v1.18.0
    │   │   ├── async-io v1.13.0 (*)
    │   │   ├── async-lock v2.8.0 (*)
    │   │   ├── crossbeam-utils v0.8.16 (*)
    │   │   ├── futures-core v0.3.28
    │   │   ├── futures-io v0.3.28
    │   │   ├── futures-lite v1.13.0 (*)
    │   │   ├── kv-log-macro v1.0.7
    │   │   │   └── log v0.4.20 (*)
    │   │   ├── log v0.4.20 (*)
    │   │   ├── memchr v2.6.1
    │   │   ├── once_cell v1.18.0
    │   │   ├── pin-project-lite v0.2.13
    │   │   ├── pin-utils v0.1.0
    │   │   └── slab v0.4.9 (*)
    │   ├── enumflags2 v0.7.7
    │   │   ├── enumflags2_derive v0.7.7 (proc-macro)
    │   │   │   ├── proc-macro2 v1.0.66
    │   │   │   │   └── unicode-ident v1.0.11
    │   │   │   ├── quote v1.0.33
    │   │   │   │   └── proc-macro2 v1.0.66 (*)
    │   │   │   └── syn v2.0.29
    │   │   │       ├── proc-macro2 v1.0.66 (*)
    │   │   │       ├── quote v1.0.33 (*)
    │   │   │       └── unicode-ident v1.0.11
    │   │   └── serde v1.0.188
    │   │       └── serde_derive v1.0.188 (proc-macro)
    │   │           ├── proc-macro2 v1.0.66 (*)
    │   │           ├── quote v1.0.33 (*)
    │   │           └── syn v2.0.29 (*)
    │   ├── futures-channel v0.3.28
    │   │   └── futures-core v0.3.28
    │   ├── futures-util v0.3.28
    │   │   ├── futures-core v0.3.28
    │   │   ├── futures-io v0.3.28
    │   │   ├── futures-macro v0.3.28 (proc-macro)
    │   │   │   ├── proc-macro2 v1.0.66 (*)
    │   │   │   ├── quote v1.0.33 (*)
    │   │   │   └── syn v2.0.29 (*)
    │   │   ├── futures-sink v0.3.28
    │   │   ├── futures-task v0.3.28
    │   │   ├── memchr v2.6.1
    │   │   ├── pin-project-lite v0.2.13
    │   │   ├── pin-utils v0.1.0
    │   │   └── slab v0.4.9 (*)
    │   ├── once_cell v1.18.0
    │   ├── rand v0.8.5
    │   │   ├── libc v0.2.147
    │   │   ├── rand_chacha v0.3.1
    │   │   │   ├── ppv-lite86 v0.2.17
    │   │   │   └── rand_core v0.6.4
    │   │   │       └── getrandom v0.2.10
    │   │   │           ├── cfg-if v1.0.0
    │   │   │           └── libc v0.2.147
    │   │   └── rand_core v0.6.4 (*)
    │   ├── serde v1.0.188 (*)
    │   ├── serde_repr v0.1.16 (proc-macro)
    │   │   ├── proc-macro2 v1.0.66 (*)
    │   │   ├── quote v1.0.33 (*)
    │   │   └── syn v2.0.29 (*)
    │   ├── url v2.4.1
    │   │   ├── form_urlencoded v1.2.0
    │   │   │   └── percent-encoding v2.3.0
    │   │   ├── idna v0.4.0
    │   │   │   ├── unicode-bidi v0.3.13
    │   │   │   └── unicode-normalization v0.1.22
    │   │   │       └── tinyvec v1.6.0
    │   │   │           └── tinyvec_macros v0.1.1
    │   │   ├── percent-encoding v2.3.0
    │   │   └── serde v1.0.188 (*)
    │   └── zbus v3.14.1
    │       ├── async-broadcast v0.5.1
    │       │   ├── event-listener v2.5.3
    │       │   └── futures-core v0.3.28
    │       ├── async-executor v1.5.1 (*)
    │       ├── async-fs v1.6.0
    │       │   ├── async-lock v2.8.0 (*)
    │       │   ├── blocking v1.3.1 (*)
    │       │   └── futures-lite v1.13.0 (*)
    │       │   [build-dependencies]
    │       │   └── autocfg v1.1.0
    │       ├── async-io v1.13.0 (*)
    │       ├── async-lock v2.8.0 (*)
    │       ├── async-recursion v1.0.4 (proc-macro)
    │       │   ├── proc-macro2 v1.0.66 (*)
    │       │   ├── quote v1.0.33 (*)
    │       │   └── syn v2.0.29 (*)
    │       ├── async-task v4.4.0
    │       ├── async-trait v0.1.73 (proc-macro)
    │       │   ├── proc-macro2 v1.0.66 (*)
    │       │   ├── quote v1.0.33 (*)
    │       │   └── syn v2.0.29 (*)
    │       ├── blocking v1.3.1 (*)
    │       ├── byteorder v1.4.3
    │       ├── derivative v2.2.0 (proc-macro)
    │       │   ├── proc-macro2 v1.0.66 (*)
    │       │   ├── quote v1.0.33 (*)
    │       │   └── syn v1.0.109
    │       │       ├── proc-macro2 v1.0.66 (*)
    │       │       ├── quote v1.0.33 (*)
    │       │       └── unicode-ident v1.0.11
    │       ├── enumflags2 v0.7.7 (*)
    │       ├── event-listener v2.5.3
    │       ├── futures-core v0.3.28
    │       ├── futures-sink v0.3.28
    │       ├── futures-util v0.3.28 (*)
    │       ├── hex v0.4.3
    │       ├── nix v0.26.4
    │       │   ├── bitflags v1.3.2
    │       │   ├── cfg-if v1.0.0
    │       │   ├── libc v0.2.147
    │       │   └── memoffset v0.7.1
    │       │       [build-dependencies]
    │       │       └── autocfg v1.1.0
    │       ├── once_cell v1.18.0
    │       ├── ordered-stream v0.2.0
    │       │   ├── futures-core v0.3.28
    │       │   └── pin-project-lite v0.2.13
    │       ├── rand v0.8.5 (*)
    │       ├── serde v1.0.188 (*)
    │       ├── serde_repr v0.1.16 (proc-macro) (*)
    │       ├── sha1 v0.10.5
    │       │   ├── cfg-if v1.0.0
    │       │   ├── cpufeatures v0.2.9
    │       │   └── digest v0.10.7
    │       │       ├── block-buffer v0.10.4
    │       │       │   └── generic-array v0.14.7
    │       │       │       └── typenum v1.16.0
    │       │       │       [build-dependencies]
    │       │       │       └── version_check v0.9.4
    │       │       └── crypto-common v0.1.6
    │       │           ├── generic-array v0.14.7 (*)
    │       │           └── typenum v1.16.0
    │       ├── static_assertions v1.1.0
    │       ├── tracing v0.1.37
    │       │   ├── cfg-if v1.0.0
    │       │   ├── pin-project-lite v0.2.13
    │       │   ├── tracing-attributes v0.1.26 (proc-macro)
    │       │   │   ├── proc-macro2 v1.0.66 (*)
    │       │   │   ├── quote v1.0.33 (*)
    │       │   │   └── syn v2.0.29 (*)
    │       │   └── tracing-core v0.1.31
    │       │       └── once_cell v1.18.0
    │       ├── xdg-home v1.0.0
    │       │   └── nix v0.26.4 (*)
    │       ├── zbus_macros v3.14.1 (proc-macro)
    │       │   ├── proc-macro-crate v1.3.1
    │       │   │   ├── once_cell v1.18.0
    │       │   │   └── toml_edit v0.19.14
    │       │   │       ├── indexmap v2.0.0
    │       │   │       │   ├── equivalent v1.0.1
    │       │   │       │   └── hashbrown v0.14.0
    │       │   │       ├── toml_datetime v0.6.3
    │       │   │       └── winnow v0.5.15
    │       │   ├── proc-macro2 v1.0.66 (*)
    │       │   ├── quote v1.0.33 (*)
    │       │   ├── regex v1.9.4
    │       │   │   ├── aho-corasick v1.0.5
    │       │   │   │   └── memchr v2.6.1
    │       │   │   ├── memchr v2.6.1
    │       │   │   ├── regex-automata v0.3.7
    │       │   │   │   ├── aho-corasick v1.0.5 (*)
    │       │   │   │   ├── memchr v2.6.1
    │       │   │   │   └── regex-syntax v0.7.5
    │       │   │   └── regex-syntax v0.7.5
    │       │   ├── syn v1.0.109 (*)
    │       │   └── zvariant_utils v1.0.1
    │       │       ├── proc-macro2 v1.0.66 (*)
    │       │       ├── quote v1.0.33 (*)
    │       │       └── syn v1.0.109 (*)
    │       ├── zbus_names v2.6.0
    │       │   ├── serde v1.0.188 (*)
    │       │   ├── static_assertions v1.1.0
    │       │   └── zvariant v3.15.0
    │       │       ├── byteorder v1.4.3
    │       │       ├── enumflags2 v0.7.7 (*)
    │       │       ├── libc v0.2.147
    │       │       ├── serde v1.0.188 (*)
    │       │       ├── static_assertions v1.1.0
    │       │       ├── url v2.4.1 (*)
    │       │       └── zvariant_derive v3.15.0 (proc-macro)
    │       │           ├── proc-macro-crate v1.3.1 (*)
    │       │           ├── proc-macro2 v1.0.66 (*)
    │       │           ├── quote v1.0.33 (*)
    │       │           ├── syn v1.0.109 (*)
    │       │           └── zvariant_utils v1.0.1 (*)
    │       └── zvariant v3.15.0 (*)
    ├── async-io v1.13.0 (*)
    ├── futures-util v0.3.28 (*)
    ├── log v0.4.20 (*)
    ├── pollster v0.3.0
    ├── raw-window-handle v0.5.2
    └── urlencoding v2.1.3

gtk is better, but it still adds more than 50 deps. Fortunately, i can already see that we have a zenity/kdialog backend coming soon which won't need any dependencies at all.
@PolyMeilex
Copy link
Owner

async-io is actually zenity backend & ashpd backend, dependency, so that could be moved under xdg feature

@coderedart
Copy link
Author

oh, does zenity need async-io too? I thought it would simply spawn a background thread and block on the commandline result.

@PolyMeilex
Copy link
Owner

It reads stdout asynchronously, but I guess this could just be moved to a blocking thread, performance does not matter for this anyway, async-io was used for that as zenity is only used for xdg-portal backend for now (for messages dialogs) and async-io is a dep of that backend already anyway.

@coderedart
Copy link
Author

zenity is only used for xdg-portal backend for now (for messages dialogs)

hmm. So, when the next version releases, is it possible to use zenity (for open/save file dialogs) without xdg-portal?

@PolyMeilex PolyMeilex mentioned this issue Aug 30, 2023
Merged
@PolyMeilex
Copy link
Owner

Once zenity backend becomes a thing someday, yes

@PolyMeilex
Copy link
Owner

Removed all the async related deps from gtk backend: #141

@PolyMeilex
Copy link
Owner

PolyMeilex commented Sep 9, 2023 •
edited
Loading

0.12 published with async deps removed from GTK feature gate

(Feel free to reopen if I missed some)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@PolyMeilex @coderedart