makefile: check and alert if GOPATH is undefined

skydive-project · Jun 3, 2020 · 0315951 · 0315951
1 parent 45fd205
commit 0315951
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/.mk/check.mk b/.mk/check.mk
@@ -1,4 +1,11 @@
 
+.PHONY: gopath
+gopath:
+
+ifndef GOPATH
+	$(error GOPATH is undefined, Skydive requires to have GOPATH defined to build)
+endif
+
 .PHONY: fmt
 fmt: genlocalfiles
 	@echo "+ $@"

diff --git a/.mk/debug.mk b/.mk/debug.mk
@@ -17,10 +17,10 @@ ifeq (${DEBUG}, true)
   export DEBUG
 endif
 
-.PHONY: debug.agent
+.PHONY: run.agent
 run.agent:
 	$(call skydive_run,agent)
 
-.PHONY: debug.analyzer
+.PHONY: run.analyzer
 run.analyzer:
 	$(call skydive_run,analyzer)
diff --git a/Makefile b/Makefile
@@ -103,6 +103,7 @@ ifeq ($(WITH_PACKETINJECT), true)
   BUILD_TAGS+=packetinject
 endif
 
+
 include .mk/ebpf.mk
 include .mk/api.mk
 include .mk/bench.mk
@@ -140,14 +141,14 @@ endef
 	$(call GOCOMPILE,build)
 
 .PHONY: build
-build: moddownload genlocalfiles .build
+build: gopath moddownload genlocalfiles .build
 
 .PHONY: .install
 .install:
 	$(call GOCOMPILE,install)
 
 .PHONY: skydive
-skydive: moddownload genlocalfiles .install
+skydive: gopath moddownload genlocalfiles .install
 
 .PHONY: skydive.clean
 skydive.clean: