-
Notifications
You must be signed in to change notification settings - Fork 21
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
Makefile is broken!!! #418
Comments
And easy way to replicate:
|
FROM docker.io/library/golang:1.17
WORKDIR /
RUN apt-get update -y
RUN apt-get install -y unzip
RUN git clone https://github.com/kubeshop/kusk-gateway.git
RUN cd kusk-gateway && make $ docker build -t test . # or `podman build -t test .`
...
go build -o bin/manager -ldflags='-X github.com/kubeshop/kusk-gateway/pkg/analytics.TelemetryToken=' cmd/manager/main.go
go build -o bin/agent -ldflags='-X github.com/kubeshop/kusk-gateway/pkg/analytics.TelemetryToken=' cmd/agent/main.go
Removing intermediate container fe0dcc0009f4
---> f91a80b17e7b
Successfully built f91a80b17e7b
Successfully tagged test:latest |
This was just an example why it failed I could install unzip but that's not the point. Two Linux boxes and one Mac can't run Even if you install unzip it will produce this error:
|
mbana
pushed a commit
that referenced
this issue
May 18, 2022
* Replace `go get` with `go install` in `Makefile`. * Tidy up `Makefile`. * Introduce `tools` in `Makefile` that downloads all the tools/dependencies required. * `DOCKER_BUILDKIT`: No point in defining this everyone - exporting and defining should be enough. For further information, see: * kubernetes-sigs/kubebuilder#2566 * kubernetes-sigs/kubebuilder#2486
mbana
added a commit
that referenced
this issue
May 18, 2022
* Replace `go get` with `go install` in `Makefile`. * Tidy up `Makefile`. * Introduce `tools` in `Makefile` that downloads all the tools/dependencies required. * `DOCKER_BUILDKIT`: No point in defining this everyone - exporting and defining should be enough. For further information, see: * kubernetes-sigs/kubebuilder#2566 * kubernetes-sigs/kubebuilder#2486
4 tasks
mbana
pushed a commit
that referenced
this issue
May 18, 2022
1. > The indentation for these variable assignments are a bit all over the place, can you align them, please? 2. > If you've tested that kustomize v4 works as expected, this comment can be removed Closes #418
mbana
pushed a commit
to kubeshop/kusk
that referenced
this issue
May 18, 2022
References or inspired by ------------------------- * <https://github.com/cli/cli/blob/trunk/pkg/cmd/version/version.go> * <https://github.com/cli/cli/blob/trunk/internal/build/build.go> * <https://github.com/cli/cli/blob/trunk/pkg/cmd/version/version_test.go> * <https://github.com/cli/cli/blob/trunk/pkg/cmd/root/root.go#L75>
mbana
added a commit
to kubeshop/kusk
that referenced
this issue
May 18, 2022
References or inspired by ------------------------- * <https://github.com/cli/cli/blob/trunk/pkg/cmd/version/version.go> * <https://github.com/cli/cli/blob/trunk/internal/build/build.go> * <https://github.com/cli/cli/blob/trunk/pkg/cmd/version/version_test.go> * <https://github.com/cli/cli/blob/trunk/pkg/cmd/root/root.go#L75>
mbana
added a commit
to kubeshop/kusk
that referenced
this issue
May 18, 2022
References or inspired by ------------------------- * <https://github.com/cli/cli/blob/trunk/pkg/cmd/version/version.go> * <https://github.com/cli/cli/blob/trunk/internal/build/build.go> * <https://github.com/cli/cli/blob/trunk/pkg/cmd/version/version_test.go> * <https://github.com/cli/cli/blob/trunk/pkg/cmd/root/root.go#L75> * <https://github.com/cli/cli/blob/trunk/Makefile> * <https://github.com/cli/cli/blob/trunk/script/build.go> (search for `ldflags`)
4 tasks
mbana
pushed a commit
to kubeshop/kusk
that referenced
this issue
May 20, 2022
Version information such as tag, revision and build date and time is built at _build_ time without having to resort to using `GO_LDFLAGS` or `-ldflags`, see: `internal/build/build.go`.
mbana
added a commit
that referenced
this issue
May 24, 2022
* Replace `go get` with `go install` in `Makefile`. * Tidy up `Makefile`. * Introduce `tools` in `Makefile` that downloads all the tools/dependencies required. * `DOCKER_BUILDKIT`: No point in defining this everyone - exporting and defining should be enough. For further information, see: * kubernetes-sigs/kubebuilder#2566 * kubernetes-sigs/kubebuilder#2486
mbana
pushed a commit
that referenced
this issue
May 24, 2022
1. > The indentation for these variable assignments are a bit all over the place, can you align them, please? 2. > If you've tested that kustomize v4 works as expected, this comment can be removed Closes #418
mbana
pushed a commit
that referenced
this issue
May 25, 2022
Fix/Cleanup `Makefile` - Closes #418: * Replace `go get` with `go install` in `Makefile`. * Tidy up `Makefile`. * Introduce `tools` in `Makefile` that downloads all the tools/dependencies required. * `DOCKER_BUILDKIT`: No point in defining this everyone - exporting and defining should be enough. For further information, see: * kubernetes-sigs/kubebuilder#2566 * kubernetes-sigs/kubebuilder#2486
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When ran on a fresh box either a docker container, linux host or macos running
make
produces this error:or in some cases
The text was updated successfully, but these errors were encountered: