mirror of
https://github.com/librespot-org/librespot.git
synced 2026-04-27 08:15:50 +03:00
[GH-ISSUE #142] Dockerfile: Install necessary dependencies for pulseaudio #106
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#106
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?
Originally created by @floli on GitHub (Feb 11, 2018).
Original GitHub issue: https://github.com/librespot-org/librespot/issues/142
Currrently, the crosscompile setup at https://github.com/librespot-org/librespot/wiki/Cross-compiling gives instruction how to compile for the portaudio backend and I have no intentions to change the default.
However, it would be nice to also add the dependencies needed to build for pulseaudio, e.g. so that
docker run -v /tmp/librespot-build:/build librespot-cross cargo build --release --target arm-unknown-linux-gnueabihf --no-default-features --features pulseaudio-backendworks. Currently there is libraries not found-lpulse -lpulse-simpleat linking.@awiouy commented on GitHub (Feb 11, 2018):
You need
libpulse-devAdd it after https://github.com/librespot-org/librespot/blob/master/contrib/Dockerfile#L25
@sashahilton00 commented on GitHub (Feb 12, 2018):
We just need to add this to the Dockerfile dependencies to compile right? If so, create a PR and will test/merge to make sure it doesn't drastically increase docker image build time, then merge.
@floli commented on GitHub (Feb 12, 2018):
Adding this line
gives me
when I add the same line for
libglib2.0-dev, I get conflictsbut honestly, I do just some cargo cult changes here, no idea what I am actually doing.
@plietar commented on GitHub (Feb 12, 2018):
Unfortunately not all debian packages support side by side installation of multiple architecures. Seems like that's the case here.
@sashahilton00 commented on GitHub (Feb 12, 2018):
You might be able to use a Docker ARG to enable the passing of a
--build-arg arch=arm64for example the conditionally install dependencies depending on which architecture you want to build...@awiouy commented on GitHub (Feb 12, 2018):
libpulse-devinstalls withdebian-sid:@floli commented on GitHub (Feb 13, 2018):
I can confirm that it build and runs just fine with your patch. I just build the armhf version for my Raspi 3 running with Raspian.
@sashahilton00 commented on GitHub (Feb 13, 2018):
For those who wish to cross-compile pulseaudio support, the patch suggested above by @awiouy works. We're not switching the build image to unstable though, so closing this for now.
@plietar commented on GitHub (Feb 14, 2018):
What's wrong with switching the build image to unstable? Debian's notion of unstable is definitely good enough for us
@awiouy commented on GitHub (Feb 14, 2018):
Workaround indicates that the problem will likely be solved by a stretch upgrade or a buster release. In other words the issue will solve itself.
Building on sid might break or break system dependencies of the compiled code.
@plietar commented on GitHub (Feb 14, 2018):
I doubt this is the sort of thing that gets backported into new stable releases, and a buster release is at least a year or two away.
Does this work on current buster?
@unimatrix27 commented on GitHub (Apr 25, 2018):
I have tried to apply this patch to cross compile for Raspberry Pi but there I still get missing -lpulse - I tried to add libpulse-dev_11.1-5+b1_armhf.deb as an additional dependency from Raspbian into the build script for PI, but that did not help. I also tried the other version that I found on raspbian.Mirror.
Any ideas?
@mfeif commented on GitHub (Jun 4, 2018):
The patch works; the compile works, but it's compiled against glibc 2.27, which is not available on my distro (current version of "DietPi") which is at 2.24
:-(