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 Docker build #237

Closed
wants to merge 1 commit into from
Closed

Fix Docker build #237

wants to merge 1 commit into from

Conversation

DRSchlaubi
Copy link

Currently the docker build creates an incompatible binary for alpine, when compiling on non musl operating systems causing a runtime error when starting the container

@jessepeterson
Copy link
Member

How about something less opinionated:

diff --git a/README.md b/README.md
index 0f765b0..aea5980 100644
--- a/README.md
+++ b/README.md
@@ -163,7 +163,7 @@ If you're not sure which SHA-256 hash (for a specific CA) to use, you can use th
 
 ```sh
 # first compile the Docker binaries
-make docker
+CGO_ENABLED=0 make docker
 
 # build the image
 docker build -t micromdm/scep:latest .

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.

2 participants