-
Notifications
You must be signed in to change notification settings - Fork 4
Conversation
Dockerfile
Outdated
@@ -0,0 +1,11 @@ | |||
FROM golang:1.10.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1.10.2
is out, might as well use that.
Dockerfile
Outdated
|
||
FROM alpine:3.7 | ||
RUN apk add --no-cache ca-certificates | ||
RUN mkdir /lib64 && ln -s /lib/libc.musl-x86_64.so.1 /lib64/ld-linux-x86-64.so.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this is necessary. The only reason gotrue
had it was because of a library that was using cgo
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had to do it because the binary was looking for that library. I guess it's related to not doing an static build
Dockerfile
Outdated
FROM golang:1.10.1 | ||
WORKDIR /go/src/github.com/netlify/twickr | ||
COPY . /go/src/github.com/netlify/twickr/ | ||
RUN go get && go build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For a static build, we need to add CGO_ENABLED=0
. We should also setup glide to vendor the dependencies. I'm happy to do that if you want.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll add the glide setup 👌
No description provided.