mirror of
https://github.com/Rigellute/spotify-tui.git
synced 2026-04-26 16:15:55 +03:00
[PR #775] [CLOSED] feat: Allow wayland builds without xcb dependency #1029
Labels
No labels
bug
enhancement
good first issue
help wanted
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/spotify-tui#1029
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/Rigellute/spotify-tui/pull/775
Author: @xanderio
Created: 3/29/2021
Status: ❌ Closed
Base:
master← Head:master📝 Commits (1)
cb86506feat: Allow wayland builds without xcb dependency📊 Changes
3 files changed (+21 additions, -1 deletions)
View changed files
📝
Cargo.toml(+6 -1)📝
src/app.rs(+13 -0)📝
src/ui/help.rs(+2 -0)📄 Description
The arbout carte is used to interact with the system clipboard. This
however imposes a dependency on xcb on the crate for Linux/Unix builds.
On a pure Wayland system this is undesirable as interaction with the
system clipboard via xcb in only possible if the user runs a xwayland
server. This cause spotify-cli to require a dependency to a x11 c lib
without providing any functionally to the user.
This change intoduces to new cargo features to spotify-cli:
x11andwayland.The
x11feature is in the default feature and will preserve thecurrent application behavior.
The
waylandfeature will disable the dependency onto arboard and use#[cfg(feature = "wayland")]attributes to disable code using thiscrate, as well as any reference from the help menu to any copy
functionally.
I currently plan to investigate how to bring this behavior back to the wayland platform.
Looking forward to your code review. :)
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.