spotify-player: 0.20.3 -> 0.20.4#377530
Conversation
|
|
|
This change looks correct to me, but I would like to know why the darwin build is failing. |
|
|
Eh.... no idea, it works on my machine?? spotify-player compiled and works on my m2, with the new lyrics sync feature as well Mic92/nixpkgs-review#368 would have probably been helpful |
|
Sorry for the late reply. I missed this pull request. I'm not sure because I just had a quick look, it seems that it requires cocoa to build spotify-player on aarch64-darwin (sandbox enabled). logs/spotify-player-aarch64-darwin.log |
0fda477 to
046fe79
Compare
|
Ah right. I now get the same error with I added the cocoa framework, which wasn't enough since I needed another fix for building mac-notification-sys, like already done here: nixpkgs/pkgs/tools/misc/starship/default.nix Lines 42 to 45 in be02d86 nixpkgs/pkgs/by-name/gh/ghostie/package.nix Lines 46 to 48 in c6e957d So I basically copied that over and now it builds on aarch64-darwin with edit: actually, not sure if the extra darwin dependency is even necessary, it builds without it and only the fix but not sure if it is needed at runtime anyway, I can't find any documentation on the darwin stuff at all and am just doing guess work |
|
momeemt
left a comment
There was a problem hiding this comment.
Thanks for the fix, it looks like it builds properly in the sandbox environment. LGTM!
bcc1b14 to
64545ae
Compare
|
Ups sorry I had to address the merge conflict caused by |
64545ae to
ab315b1
Compare
https://github.com/aome510/spotify-player/releases/tag/v0.20.4
lyric_finder feature was exchanged with lyric_finder crate
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.