-
Notifications
You must be signed in to change notification settings - Fork 95
Fix regressions on Wayland due to ashpd upgrade #255
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
Conversation
The upgrade to ashpd 0.11 effectively disabled the `wayland' feature, which broke rfd on Wayland. This patch reenables the feature.
|
What exactly do you mean by "broke rfd on Wayland"? 👀 |
|
I mean that Wayland-specific things aren't working. I discovered it specifically because the file chooser window didn't have a parent-child relationship to the application ( I haven't really investigated what other things broke, just noticed that the whole |
|
Thanks! |
|
@PolyMeilex Would you be so kind and release 0.15.4 with this fix? I think it's pretty important. Thank you! |
|
Done, should be up. Thanks again 🎉 |
This fixes regressions on Wayland: PolyMeilex/rfd#255
|
Could this be made optional with a feature flag in the next minor version bump? It's a bunch of extra dependencies and breaks CI when |
Came here to say this, dependabot's minor update broke CI due to Wayland dependencies not being installed |
It's actually the previous patch release that broke things. This one reverts it to how it was before. |
|
Perhaps the best way forward is to just enable dynamic linkage via To test that, you can just plop this into your cargo.toml and it should make the dependency optional. wayland-backend = { version = "0.3.10", features = ["dlopen"] |
The upgrade to ashpd 0.11 effectively disabled the `wayland' feature, which broke some rfd features on Wayland. This patch reenables the feature.
See #240 (comment)