Skip to content

Conversation

@sodagunz
Copy link
Contributor

@sodagunz sodagunz commented Aug 25, 2025

  • Bumps librespot to 0.7.0
  • Changes worker authentication to use login5
  • Solves minor API changes
  • Adds alsa-lib to nix shell

Describe your changes

These PR is only about bumping librespot to 0.7.0, as some changes in spotify's API caused issues in any project using librespot <=0.6.0
ncspot is basically bricked without these changes.

Credits to blueware-dev, who had released a temporary fix pointing to librespot-dev branch, and did the bulk of the work here by providing a workaround.

Issue ticket number and link

Should address these tickets:

Checklist before requesting a review

  • Documentation was updated (i.e. due to changes in keybindings, commands, etc.)
    • No changes in bindings or commands
  • Changelog was updated with relevant user-facing changes (eg. not dependency updates,
    not performance improvements, etc.)
    • No user facing changes

* Bumps librespot to 0.7.0
* Changes worker authentication to use login5
* Solves minor API changes
* Adds alsa-lib to nix shell
@hrkfdn
Copy link
Owner

hrkfdn commented Aug 25, 2025

Thank you! ❤️ Could you add a change log entry to indicate that this fixes the current problems? I don't have access to a computer right now, but I'm planning to draft a release later today or tomorrow the latest.

@sodagunz
Copy link
Contributor Author

Sure, it looks like we have a new dependency on alsa-lib, not sure how desirable that is. I'll fix the CI actions if that's acceptable, or look for a workaround after work (18:30 UTC+2) if that's not acceptable.

And my pleasure, ncspot is likely my favorite TUI ❤️

Allow users to explicitly specify Librespot audio backend. This way we can also
drop the ALSA dependencies again.
@hrkfdn hrkfdn merged commit e5e4c97 into hrkfdn:main Aug 25, 2025
6 checks passed
@hrkfdn
Copy link
Owner

hrkfdn commented Aug 25, 2025

Merged, thank you! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants