Skip to content

Commit

Permalink
Revert jaegertracing#5482 - ppc64le is not yet supported by delve
Browse files Browse the repository at this point in the history
Signed-off-by: Yuri Shkuro <github@ysh.us>
  • Loading branch information
yurishkuro committed May 26, 2024
1 parent 8d12410 commit 6bf878e
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions docker/debug/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,9 @@ FROM golang:1.22.3-alpine AS build
ARG TARGETARCH
ENV GOPATH /go
RUN apk add --update --no-cache ca-certificates make git build-base mailcap
# Remove once go-delve adds support for s390x (https://github.com/go-delve/delve/issues/2883)
RUN if [[ "$TARGETARCH" == "s390x" ]] ; then \
# TODO: Remove s390x once go-delve adds support for it (https://github.com/go-delve/delve/issues/2883)
# TODO: Remove ppc64le once support is released (https://github.com/go-delve/delve/issues/1564) - not yet as of delve@v1.22.1
RUN if [[ "$TARGETARCH" == "s390x" || "$TARGETARCH" == "ppc64le" ]] ; then \
touch /go/bin/dlv; \
else \
go install github.com/go-delve/delve/cmd/dlv@latest; \
Expand Down

0 comments on commit 6bf878e

Please sign in to comment.