mirror of
https://github.com/librespot-org/librespot.git
synced 2026-04-27 08:15:50 +03:00
[PR #964] [MERGED] Update to GStreamer bindings version 0.18 and various cleanups to the GStreamer audio backend #1165
Labels
No labels
A-Alsa
SpotifyAPI
Tokio 1.0
audio
bug
can't reproduce
compilation
dependencies
duplicate
enhancement
good first issue
help wanted
high priority
imported
imported
invalid
new api
pull-request
question
reverse engineering
wiki
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/librespot#1165
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/librespot-org/librespot/pull/964
Author: @sdroege
Created: 2/3/2022
Status: ✅ Merged
Merged: 2/13/2022
Merged by: @roderickvd
Base:
new-api← Head:gst-0.18📝 Commits (10+)
6a23594Update GStreamer backend to 0.187db566dDon't manually go through all intermediate states when shutting down the GStreamer backendfa07202Don't initialize GStreamer twice2ede631Use less stringly-typed API for configuring the appsrc88ea5d1Create our own main context instead of stealing the default one0fcbdcdCreate GStreamer pipeline more explicitly instead of going via strings for everything1bd3bf5Add an audioresample element before the sink in case the sink doesn't support the sample rate950f20fRemove unnecessary as_bytes() call3391c8eRun the GStreamer backend through rustfmt7d0514eUse a GStreamer bus sync handler instead of spawning a new thread with a mainloop📊 Changes
5 files changed (+174 additions, -161 deletions)
View changed files
📝
.github/workflows/test.yml(+3 -3)📝
COMPILING.md(+1 -1)📝
Cargo.lock(+77 -81)📝
playback/Cargo.toml(+5 -4)📝
playback/src/audio_backend/gstreamer.rs(+88 -72)📄 Description
See individual commits for details.
Previous discussion here: https://github.com/librespot-org/librespot/pull/961
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.