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.

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

profiles: ytmdesktop: add redirect & whitelist /opt/ytmdesktop #6666

Merged
merged 2 commits into from
Feb 26, 2025

Conversation

kmk3
Copy link
Collaborator

@kmk3 kmk3 commented Feb 25, 2025

Apparently the main binary has been renamed from ytmdesktop to
youtube-music-desktop-app[1]:

$ pacman -Qlq ytmdesktop
[...]
/opt/ytmdesktop/youtube-music-desktop-app
/usr/bin/youtube-music-desktop-app
/usr/share/applications/ytmdesktop.desktop

So add a redirect for it.

Fixes #6662.

[1] #6662 (comment)

Reported-by: @Dieterbe


ytmdesktop: whitelist /opt/ytmdesktop

See also commit 1759055 ("profiles: exchange private-opt with a
whitelist (#6021)", 2023-10-18).

@kmk3 kmk3 requested a review from rusty-snake February 25, 2025 21:03
Copy link
Collaborator

@rusty-snake rusty-snake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

firecfg.config

kmk3 added 2 commits February 25, 2025 18:14
Apparently the main binary has been renamed from `ytmdesktop` to
`youtube-music-desktop-app`[1]:

    $ pacman -Qlq ytmdesktop
    [...]
    /opt/ytmdesktop/youtube-music-desktop-app
    /usr/bin/youtube-music-desktop-app
    /usr/share/applications/ytmdesktop.desktop

So add a redirect for it.

Fixes netblue30#6662.

[1] netblue30#6662 (comment)

Reported-by: @Dieterbe
From [1]:

    $ pacman -Qlq ytmdesktop
    [...]
    /opt/ytmdesktop/youtube-music-desktop-app
    /usr/bin/youtube-music-desktop-app
    /usr/share/applications/ytmdesktop.desktop

See also commit 1759055 ("profiles: exchange private-opt with a
whitelist (netblue30#6021)", 2023-10-18).

[1] netblue30#6662 (comment)
@kmk3 kmk3 force-pushed the ytmdesktop-add-redirect branch from 51aeb21 to 985cb43 Compare February 25, 2025 21:15
@kmk3
Copy link
Collaborator Author

kmk3 commented Feb 25, 2025

firecfg.config

Good catch; fixed.

@kmk3 kmk3 merged commit e71f117 into netblue30:master Feb 26, 2025
8 checks passed
@kmk3 kmk3 deleted the ytmdesktop-add-redirect branch February 26, 2025 12:06
@Dieterbe
Copy link
Contributor

thank you ! :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

ytmdesktop: new version renames the binary to youtube-music-desktop-app
3 participants