Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issues with our docker builds #250

Merged
merged 2 commits into from
Oct 1, 2021
Merged

Fix issues with our docker builds #250

merged 2 commits into from
Oct 1, 2021

Conversation

gcampax
Copy link
Contributor

@gcampax gcampax commented Aug 5, 2021

Related to #201 and #228

The spotify skill uses wget to download spotifyd, and it expects
wget will be there.

Addresses #201
@gcampax gcampax added the P1 We're working on it right now label Aug 5, 2021
@gcampax gcampax force-pushed the wip/fix-docker-issues branch 5 times, most recently from 946b2d8 to 9fe69fc Compare August 6, 2021 20:29
While our application to Docker Open Source is under review, builds
on DockerHub are not available. Instead, set up travis to build
our docker images.

The full build is done with "docker buildx", which in theory lets
us build images for other architectures. In practice, the normal
build (changing the base image to an arch specific image) works
for ARM but fails in "docker buildx", and I'm not sure exactly why.
@gcampax gcampax requested a review from nrser August 6, 2021 20:56
@gcampax gcampax merged commit 5ca06bf into master Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 We're working on it right now
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant