Skip to content

deps(librespot): bump librespot to pre 0.8.0#56

Open
loloster wants to merge 4 commits into
GuillemCastro:masterfrom
loloster:bump-librespot-0.8.0
Open

deps(librespot): bump librespot to pre 0.8.0#56
loloster wants to merge 4 commits into
GuillemCastro:masterfrom
loloster:bump-librespot-0.8.0

Conversation

@loloster

Copy link
Copy Markdown

Due to the recent changes released by Spotify, metadata retreival was failing,
so downloads. Through librespot fix #1622
librespot dev branch (pre 0.8.0) have been updated.
But since that dev branch have introduced some breaking changes, spotify-dl
have to also be updated (see track.rs)

flxo and others added 4 commits September 25, 2025 20:14
Due to the recent changes released by Spotify no metadata retrieval is possible.
`librespot` 0.7.1 adds these needed changes.
Due to the recent changes released by Spotify librespot dev branch
have been updated but since that branch have some breaking changes,
spotify-dl have to also be updated (see track.rs)
@brainslush

brainslush commented Nov 19, 2025

Copy link
Copy Markdown

I was about to write that you have a borrow issue. I had an old state.
But, I see that your latest commit already fixes that.
Otherwise It seem to work fine.

Comment thread Cargo.toml
structopt = { version = "0.3", default-features = false }
indicatif = "0.18"
librespot = { version = "0.6.0", default-features = false }
librespot = { version = "0.8.0" }

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In order to keep librespot’s dependency set small, I suggest keeping default-features = false and explicitly selecting the TLS backend, as the build error suggests when it’s not set. If we don’t, librespot pulls in extra audio playback dependencies such as ALSA.

Suggested change
librespot = { version = "0.8.0" }
librespot = { version = "0.8.0", default-features = false, features = ["rustls-tls-native-roots"] }

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.

4 participants