Skip to content

Conversation

@kjarosh
Copy link
Contributor

@kjarosh kjarosh commented Jul 7, 2025

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)

The upgrade to ashpd 0.11 effectively disabled the `wayland' feature,
which broke rfd on Wayland. This patch reenables the feature.
@torokati44
Copy link
Contributor

What exactly do you mean by "broke rfd on Wayland"? 👀

@kjarosh
Copy link
Contributor Author

kjarosh commented Jul 7, 2025

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 (set_parent stopped working).

I haven't really investigated what other things broke, just noticed that the whole wayland feature was disabled in ashpd and all Wayland dependencies were removed from rfd.

@PolyMeilex PolyMeilex merged commit d9d3188 into PolyMeilex:master Jul 9, 2025
13 checks passed
@PolyMeilex
Copy link
Owner

Thanks!

@kjarosh kjarosh deleted the ashpd-wayland branch July 9, 2025 18:16
@kjarosh
Copy link
Contributor Author

kjarosh commented Jul 10, 2025

@PolyMeilex Would you be so kind and release 0.15.4 with this fix? I think it's pretty important. Thank you!

@PolyMeilex
Copy link
Owner

PolyMeilex commented Jul 12, 2025

Done, should be up. Thanks again 🎉

kjarosh added a commit to kjarosh/ruffle that referenced this pull request Jul 12, 2025
This fixes regressions on Wayland: PolyMeilex/rfd#255
@edwloef
Copy link
Contributor

edwloef commented Jul 13, 2025

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 libwayland-client.so isn't installed.

@dealloc
Copy link

dealloc commented Jul 14, 2025

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 libwayland-client.so isn't installed.

Came here to say this, dependabot's minor update broke CI due to Wayland dependencies not being installed

@kjarosh
Copy link
Contributor Author

kjarosh commented Jul 14, 2025

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.

@PolyMeilex
Copy link
Owner

PolyMeilex commented Jul 16, 2025

Perhaps the best way forward is to just enable dynamic linkage via dlopen feature of wayland-rs.

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"]

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.

5 participants