Skip to content

Add build target which creates debug image #110

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.

Sign up for GitHub

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

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Add build target which creates debug image #110

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
43ca1df
build debug image
replay Dec 14, 2023
befa19f
only arm
replay Dec 14, 2023
1415066
add target to publish debug image
replay Dec 14, 2023
a53a136
update title
replay Dec 14, 2023
b6a6cb3
correct tag prefix
replay Dec 14, 2023
04a2d3a
same repo different tag
replay Dec 14, 2023
0641cd1
fix dlv path
replay Dec 14, 2023
714108c
fix delve parameters
replay Dec 14, 2023
21d43ba
fix build flags
replay Dec 14, 2023
c1d94f5
go install dlv like normal
replay Dec 14, 2023
1d4005d
update alpine
replay Dec 15, 2023
97d27f4
comment
replay Dec 15, 2023
bed708e
separate build targets per arch for local builds
replay Dec 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions Dockerfile.delve
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM golang:1.21-alpine3.18 AS build
FROM golang:1.21-alpine3.19 AS build
Copy link
Collaborator

@andyasp andyasp Dec 15, 2023 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can't update to alpine 3.19 yet, see #97.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The above has been resolved now, but the main Dockerfile now compiles in bookworm. I don't think that difference matters here besides consistency.


ARG TARGETOS
ARG TARGETARCH
Expand All @@ -12,7 +12,7 @@ RUN go install github.com/go-delve/delve/cmd/[email protected]
RUN /go/bin/dlv -h
RUN GOOS=${TARGETOS} GOARCH=${TARGETARCH} make ${BUILDTARGET}

FROM alpine:3.18
FROM alpine:3.19
RUN apk add --no-cache ca-certificates

COPY --from=build /go/bin/dlv /bin/dlv
Expand Down