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

spotifyd does not build reproducibly #1167

Closed
bmwiedemann opened this issue Feb 17, 2023 · 1 comment · Fixed by #1182
Closed

spotifyd does not build reproducibly #1167

bmwiedemann opened this issue Feb 17, 2023 · 1 comment · Fixed by #1182
Labels
enhancement A new feature that would improve Spotifyd
Milestone

Comments

@bmwiedemann
Copy link

Description
While working on reproducible builds for openSUSE, I found that
our spotifyd package varied in each build.

To Reproduce

  1. build spotifyd binaries twice from scratch
  2. cmp spotifyd{1,2}

Expected behavior
binaries should be able to produce bit-identical binaries.

Logs

Click to show logs
--- old /usr/bin/spotifyd (objdump)
+++ new /usr/bin/spotifyd (objdump)
@@ -631764,7 +631764,7 @@
  9a5020 60617379 6e632066 6e602072 6573756d  `async fn` resum
  9a5030 65642061 66746572 20636f6d 706c6574  ed after complet
  9a5040 696f6e6c 69627265 73706f74 5f5f554e  ionlibrespot__UN
- 9a5050 4b4e4f57 4e437a31 4a6f4746 6373706f  KNOWNCz1JoGFcspo
+ 9a5050 4b4e4f57 4e5a6259 30435948 6f73706f  KNOWNZbY0CYHospo
  9a5060 74696679 646c6962 72657370 6f742d55  tifydlibrespot-U
  9a5070 4e4b4e4f 574e454f 46526563 65697665  NKNOWNEOFReceive
  9a5080 6420696e 76616c69 64207061 636b6574  d invalid packet

Versions

  • OS: openSUSE-Tumbleweed-20230215
  • Spotifyd: 0.3.3
  • cargo: 1.67

Additional context

This is fixed with
librespot-org/librespot@9de1f38
so using librespot-core 0.4+ instead of 0.2.0 solves this problem.

@bmwiedemann bmwiedemann added the bug A functionality or parts of a program that do not work as intended label Feb 17, 2023
@eladyn eladyn added enhancement A new feature that would improve Spotifyd and removed bug A functionality or parts of a program that do not work as intended labels Feb 18, 2023
@eladyn
Copy link
Member

eladyn commented Feb 18, 2023

Thanks for raising this. Guess, we'll have to keep that open, until someone gets around to doing that (presumably massive) update. 😅

I might be able to start tackling that in a few weeks, when I have more time. Still not sure, though, whether it makes sense to put in the work for 0.4, while 0.5 is already in the works with some very promising features. Probably still better, to not let the gap grow further between librespot upstream and our used version.

@slondr slondr linked a pull request Mar 9, 2023 that will close this issue
@slondr slondr added this to the v0.3.5 milestone Mar 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A new feature that would improve Spotifyd
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants