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
Spotify 1.0.96.181 -> 1.1.10.546 #72596
Conversation
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.
Not sure if we actually want to do the upgrade. When starting spotify from the CLI (by running ./result/bin/spotify
) it segfaults right after playing the first song:
./result/bin/spotify: /nix/store/dbaf5pq3lrdmwzmkdr4k60ikglvl01rv-curl-7.66.0/lib/libcurl-gnutls.so.4: no version information available (required by ./result/bin/spotify)
/nix/store/sg41pfc2xxbgnq8m9jnd07jff5wc56kw-spotify-1.1.10.546.ge08ef575-19/share/spotify/.spotify-wrapped: /nix/store/dbaf5pq3lrdmwzmkdr4k60ikglvl01rv-curl-7.66.0/lib/libcurl-gnutls.so.4: no version information available (required by /nix/store/sg41pfc2xxbgnq8m9jnd07jff5wc56kw-spotify-1.1.10.546.ge08ef575-19/share/spotify/.spotify-wrapped)
/proc/self/exe: /nix/store/dbaf5pq3lrdmwzmkdr4k60ikglvl01rv-curl-7.66.0/lib/libcurl-gnutls.so.4: no version information available (required by /proc/self/exe)
[1102/213839.640465:ERROR:buffer_manager.cc(488)] [.DisplayCompositor]GL ERROR :GL_INVALID_OPERATION : glBufferData: <- error from previous GL command
[1] 26004 segmentation fault (core dumped) ./result/bin/spotify
Interesting, I hadn't noticed that. Looks like the issue is somewhat known: https://community.spotify.com/t5/Desktop-Linux/GL-Error-on-Ubuntu-18-04/td-p/4642580 |
It appears to be a problem with libcurl/gnutls: https://community.spotify.com/t5/Desktop-Linux/Ubuntu-19-04-deb-package-segfault/m-p/4764926/highlight/true#M18393 |
When I run
Oddly enough, running (On another note, |
Running
Diffing the
Since that was a new dependency, maybe that's where the problem is coming from? (As you can probably tell by now, I'm still a noob with all this, any hints are appreciated) |
This is the ldd ouput from an ubuntu machine running spotify
As you can see, there are quite some libraries not linked in the nixos version. I'm not sure if that is an issue though... but since we're getting a segfault, it may be? |
After reading through #65602, I got it to work with pulseaudio with this: #65602 (comment), effectivly just adding My pulseaudio config is this:
I could somewhat get it to work with alsa, but not good. I did not get any audio from other apps when running spotify with that... and it only started playing when I stopped everything else which produced audio. Using pulseaudio is a more stable experience for me. @timokau @angristan @ajs124 Could you test if this works for you? |
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.
Works for me. Would like to have at least another tester before merging though.
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.
Works for me as well. Probably no use in trying to support alsa when upstream clearly doesn't care for it.
Thanks for tackling this @kolaente! |
Should I backport this to 19.09? |
No, unless I'm missing some specific reason to backport. We mostly just backport fixes and maybe minor releases. Normal updates always have the potential of breaking stuff, which goes against the idea of stable channels. |
@timokau Makes sense, thanks for the explaination! |
at-spi2-core
is required for the new dependency onlibatspi.so.0
.Motivation for this change
Spotify minor version update
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 @edolstra @ftrvxmtrx @sheenobu @laMudri @timokau