Skip to content

Commit

Permalink
fix: review comments to not to add -s flag
Browse files Browse the repository at this point in the history
  • Loading branch information
abhimanyu003 authored Oct 1, 2024
1 parent e07189b commit 6931488
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 14 deletions.
2 changes: 1 addition & 1 deletion components/tls/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ lint: ${.GOLANGCILINT_PATH}/golangci-lint

.PHONY: build
build: test
go build -trimpath -ldflags="-s -w" ./pkg/...
go build -trimpath -ldflags="-w" ./pkg/...

.PHONY: licenses/dep.txt
licenses/dep.txt:
Expand Down
2 changes: 1 addition & 1 deletion hodometer/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ lint: ${.GOLANGCILINT_PATH}/golangci-lint
.PHONY: build-hodometer
build-hodometer: test-hodometer
$(GO_ENV) \
go build -trimpath -ldflags="-s -w" \
go build -trimpath -ldflags="-w" \
-o bin/hodometer \
-ldflags=" \
-X '$(VERSION_PACKAGE).BuildVersion=$(BUILD_VERSION)' \
Expand Down
2 changes: 1 addition & 1 deletion operator/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ COPY operator/pkg/ pkg/
COPY operator/main.go main.go

# Build
RUN CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -trimpath -ldflags="-s -w" -a -o manager main.go
RUN CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -trimpath -ldflags="-w" -a -o manager main.go

FROM registry.access.redhat.com/ubi9/ubi-micro:9.4-15
WORKDIR /
Expand Down
8 changes: 4 additions & 4 deletions operator/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -88,20 +88,20 @@ test: manifests generate fmt vet envtest ## Run tests.

.PHONY: build
build: generate fmt vet ## Build manager binary.
go build -trimpath -ldflags="-s -w" -o bin/manager main.go
go build -trimpath -ldflags="-w" -o bin/manager main.go

.PHONY: run
run: manifests generate fmt vet ## Run a controller from your host.
go run ./main.go

build-seldon: generate fmt vet
go build -trimpath -ldflags="-s -w" -o bin/seldon -v ./cmd/seldon
go build -trimpath -ldflags="-w" -o bin/seldon -v ./cmd/seldon

build-seldon-arm: generate fmt vet
GOOS=darwin GOARCH=arm64 go build -trimpath -ldflags="-s -w" -tags dynamic -o bin/seldon -v ./cmd/seldon
GOOS=darwin GOARCH=arm64 go build -trimpath -ldflags="-w" -tags dynamic -o bin/seldon -v ./cmd/seldon

build-seldon-docgen:
go build -trimpath -ldflags="-s -w" -o bin/seldon-gendocs -v ./cmd/seldon/docs
go build -trimpath -ldflags="-w" -o bin/seldon-gendocs -v ./cmd/seldon/docs

generate-cli-docs:
./bin/seldon-gendocs --out ../docs/source/contents/cli/docs
Expand Down
14 changes: 7 additions & 7 deletions scheduler/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -32,31 +32,31 @@ GO_LDFLAGS := -s -w $(patsubst %,-X %, $(GO_BUILD_VARS))

.PHONY: build-scheduler
build-scheduler: test-go
go build -trimpath -ldflags="-s -w" -buildvcs=false -o bin/scheduler ./cmd/scheduler
go build -trimpath -ldflags="-w" -buildvcs=false -o bin/scheduler ./cmd/scheduler

.PHONY: build-proxy
build-proxy: test-go
CGO_ENABLED=0 go build -trimpath -ldflags="-s -w" -o bin/proxy ./cmd/proxy
CGO_ENABLED=0 go build -trimpath -ldflags="-w" -o bin/proxy ./cmd/proxy

.PHONY: build-agent
build-agent: test-go
CGO_ENABLED=0 go build -trimpath -ldflags="-s -w" -buildvcs=false -o bin/agent ./cmd/agent
CGO_ENABLED=0 go build -trimpath -ldflags="-w" -buildvcs=false -o bin/agent ./cmd/agent

.PHONY: build-modelgateway
build-modelgateway: test-go
go build -trimpath -ldflags="-s -w" -o bin/modelgateway -v ./cmd/modelgateway
go build -trimpath -ldflags="-w" -o bin/modelgateway -v ./cmd/modelgateway

.PHONY: build-pipelinegateway
build-pipelinegateway: test-go
go build -trimpath -ldflags="-s -w" -o bin/pipelinegateway -v ./cmd/pipelinegateway
go build -trimpath -ldflags="-w" -o bin/pipelinegateway -v ./cmd/pipelinegateway

.PHONY: build-dataflow-producer
build-dataflow-producer: test-jvm
go build -trimpath -ldflags="-s -w" -o data-flow/scripts/bin/producer ./data-flow/scripts/producer.go ./data-flow/scripts/common.go
go build -trimpath -ldflags="-w" -o data-flow/scripts/bin/producer ./data-flow/scripts/producer.go ./data-flow/scripts/common.go

.PHONY: build-dataflow-consumer
build-dataflow-consumer:
go build -trimpath -ldflags="-s -w" -o data-flow/scripts/bin/consumer ./data-flow/scripts/consumer.go ./data-flow/scripts/common.go
go build -trimpath -ldflags="-w" -o data-flow/scripts/bin/consumer ./data-flow/scripts/consumer.go ./data-flow/scripts/common.go

.PHONY: build-dataflow-engine
build-dataflow-engine:
Expand Down

0 comments on commit 6931488

Please sign in to comment.