mirror of
https://github.com/librespot-org/librespot.git
synced 2026-04-27 08:15:50 +03:00
[GH-ISSUE #243] Ignored contrib folder causes Raspberry Pi build to fail #164
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#164
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 @bricedp on GitHub (Aug 17, 2018).
Original GitHub issue: https://github.com/librespot-org/librespot/issues/243
As a result of the recently merged #237, the contrib folder is now ignored by Docker. While most builds do not use this folder, the Raspberry Pi build uses the file docker-build-pi-armv6hf.sh, which is in that folder. Running the Docker image fails like so:
When inspecting the image, the file is (as expected) not found:
Either the contrib folder should be removed from the .dockerignore file, or the build script should be moved elsewhere in the directory structure so that it can be included in the built image. If it is moved, the corresponding instructions in the Dockerfile need to be updated as well.
@bricedp commented on GitHub (Aug 17, 2018):
Similarly, the default Dockerfile command (
CMD ["/src/contrib/docker-build.sh"]) uses the file docker-build.sh, which is also in the contrib directory.@bilak commented on GitHub (Aug 30, 2018):
this commit just broken the build. Remove contrib from .dockerignore and you are good to go.
@ComlOnline commented on GitHub (Sep 1, 2018):
Sorry for not catching that before merging
@sashahilton00 commented on GitHub (Nov 3, 2018):
will revert that, one sec
@sashahilton00 commented on GitHub (Nov 3, 2018):
Removed contrib from
.dockerignoreingithub.com/librespot-org/librespot@0331a7f592, should now work.