Skip to content
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

chore: unite the bin cache dir for all projects in go.work #609

Merged
merged 9 commits into from
Jul 2, 2024
5 changes: 5 additions & 0 deletions common.mk
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,11 @@ SHELL = /bin/bash
OS = $(shell uname)
IN_CI ?=

ROOT_DIR := $(shell dirname $(realpath $(firstword $(MAKEFILE_LIST))))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

linke to #609 (comment)

Here is my solution:)

@spacewander

LOCALBIN := $(ROOT_DIR)/bin
$(LOCALBIN):
@mkdir -p $(LOCALBIN)

TARGET_SO = libgolang.so
PROJECT_NAME = mosn.io/htnn
# Both images use glibc 2.31. Ensure libc in the images match each other.
Expand Down
6 changes: 3 additions & 3 deletions controller/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -89,11 +89,11 @@ KIND ?= kind
##@ Build Dependencies

## Location to install dependencies to
LOCALBIN ?= $(shell cd .. && pwd)/bin
#LOCALBIN ?= $(shell cd .. && pwd)/bin
# Remember to remove tools downloaded into bin directory manually before updating them.
# If they need to be updated frequently, we can consider to store them in the `Dockerfile.dev`.
$(LOCALBIN):
mkdir -p $(LOCALBIN)
#$(LOCALBIN):
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just comment out the above code temporarily.

# mkdir -p $(LOCALBIN)

## Tool Binaries
KUBECTL ?= kubectl
Expand Down
Loading