-
-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
youtube-music: 0.17.0 -> 0.18.0 #205418
youtube-music: 0.17.0 -> 0.18.0 #205418
Conversation
Nixpkgs-review on
The resulting binary could be opened and used to play the intros of some random songs and allowed navigation within. Seems to be functional. Activating/deactivating of plugins seem to have no effect other than seeing corresponding log lines on application start, this behaviour is reproducible with the prev. version. |
Did you check that mpris is working correctly? |
mpris works as expected (im using mpris-proxy if thats relavant) |
I have no clue what mpris means. So I have not checked it consciously. |
I use playerctl for that. I'll approve the modification but I can't merge |
I confirm mpris is now broken on my side |
@rubyowo I am not sure it is youtube-music which is responding here. On my machine, it appears as "youtube-music" not "chromium". If I open the website on my chromium browser it appears and works the same as you. |
it shows up as chromium.instance2 here |
how do you start it? do you run wayland? 1.17 was working correctly |
I run wayland (Hyprland) and i just launch it from its .desktop file, if i launch it from the terminal it also works |
I run sway. It is strange because the name is the one used by the binary to register itself in dbus (you can check it with dfeet). |
ok. found. I had "shortcuts" enabled in the app. This seems to be based on the faulty npm libs mentioned in this bug deactivating it make the system to use the default system in chromium |
Description of changes
https://github.com/th-ch/youtube-music/blob/master/changelog.md#v1180
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes