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
spotifyd: 0.2.19 -> 0.2.20 (mpris enabled) #73042
Conversation
Co-Authored-By: Alexey Shmalko <rasen.dubi@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I get the following error:
[ERROR] Caught panic with message: Failed to initialize DBus connection: D-Bus error: Unable to autolaunch a dbus-daemon without a $DISPLAY for X11 (org.freedesktop.DBus.Error.NotSupported)
Use withDBUS ? false
instead of withDBUS ? true
?
With this version bump my
The fix was to clear out the EDIT: Or perhaps we should ask upstream to fix this problem? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The cache issue is not Nix-specific, so I would go for upstream.
The dbus start issue seems to be caused by spotifyd not running with X11. Not sure how much of an issue that is as I don't use spotifyd.
@MtP76 what do you think?
Will do. If upstream fixes this issue then I would vote for skipping the 0.2.20 version, so as to not break the spotifyd service for people.
Definitely an issue since one use case (for example, my use case) is running spotifyd as a headless Spotify Connect client. |
|
After clearing the cache I cannot reproduce the cache problem (see Spotifyd/spotifyd#433 for more details). Upstream will not fix the problem unless it happens more often. I guess we will have to accept that this version bump might break the spotifyd service? I would still need to see the DBUS without X11 problem solved before approving this pull request, though. |
As a quick fix, we could encode spotifyd version in the cache directory name? Instead of |
That would indeed solve the problem. But there are the obvious disadvantages to clearing the cache: songs need to be recached and credentials need to be added again. If the problem I ran into is rare enough then perhaps such a quickfix is not worth it? I do not know what to prefer here. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @