[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[xen staging] golang/xenlight: Get rid of GOPATH-based build artefacts

commit b84c47221d2273b7d84866fce9c511ec3ba5abb8
Author:     George Dunlap <george.dunlap@xxxxxxxxxx>
AuthorDate: Thu May 28 12:20:54 2020 +0100
Commit:     George Dunlap <george.dunlap@xxxxxxxxxx>
CommitDate: Thu May 28 12:20:54 2020 +0100

    golang/xenlight: Get rid of GOPATH-based build artefacts
    The original build setup used a "fake GOPATH" in tools/golang to test
    the mechanism of building from go package files installed on a
    filesystem.  With the move to modules, this isn't necessary, and leads
    to potentially confusing directories being created.  (I.e., it might
    not be obvious that files under tools/golang/src shouldn't be edited.)
    Get rid of the code that creates this (now unused) intermediate
    directory.  Add direct dependencies from 'build' onto the source
    Signed-off-by: George Dunlap <george.dunlap@xxxxxxxxxx>
    Reviewed-by: Nick Rosbrook <rosbrookn@xxxxxxxxxxxx>
 tools/Rules.mk                 |  1 -
 tools/golang/Makefile          | 10 ----------
 tools/golang/xenlight/Makefile | 22 +++++-----------------
 3 files changed, 5 insertions(+), 28 deletions(-)

diff --git a/tools/Rules.mk b/tools/Rules.mk
index 59c72e7a88..76acaef988 100644
--- a/tools/Rules.mk
+++ b/tools/Rules.mk
@@ -35,7 +35,6 @@ XENSTORE_XENSTORED ?= y
 debug ?= y
 debug_symbols ?= $(debug)
-XEN_GOPATH        = $(XEN_ROOT)/tools/golang
 XEN_GOCODE_URL    = golang.xenproject.org
 ifeq ($(debug_symbols),y)
diff --git a/tools/golang/Makefile b/tools/golang/Makefile
index aba11ebc39..b022e2c5a3 100644
--- a/tools/golang/Makefile
+++ b/tools/golang/Makefile
@@ -1,16 +1,6 @@
 include $(XEN_ROOT)/tools/Rules.mk
-# In order to link against a package in Go, the package must live in a
-# directory tree in the way that Go expects.  To make this possible,
-# there must be a directory such that we can set GOPATH=${dir}, and
-# the package will be under $GOPATH/src/${full-package-path}.
-# So we set XEN_GOPATH to $XEN_ROOT/tools/golang.  The xenlight
-# "package build" directory ($PWD/xenlight) will create the "package
-# source" directory in the proper place.  Go programs can use this
-# package by setting GOPATH=$(XEN_GOPATH).
 SUBDIRS-y = xenlight
 .PHONY: build all
diff --git a/tools/golang/xenlight/Makefile b/tools/golang/xenlight/Makefile
index 8ab4cb5665..eac9dbf12a 100644
--- a/tools/golang/xenlight/Makefile
+++ b/tools/golang/xenlight/Makefile
@@ -4,8 +4,7 @@ include $(XEN_ROOT)/tools/Rules.mk
 # Standing boldly against convention, we insist on installing the
 # package source under $(prefix)/share/gocode
 GOCODE_DIR ?= $(prefix)/share/gocode/
-GOXL_PKG_DIR = /src/$(XEN_GOCODE_URL)/xenlight/
 GO ?= go
@@ -14,17 +13,8 @@ LIBXL_SRC_DIR = ../../libxl
 .PHONY: all
 all: build
-.PHONY: package
-package: $(XEN_GOPATH)$(GOXL_PKG_DIR)
 GOXL_GEN_FILES = types.gen.go helpers.gen.go
-$(XEN_GOPATH)/src/$(XEN_GOCODE_URL)/xenlight/: xenlight.go $(GOXL_GEN_FILES)
-       $(INSTALL_DATA) xenlight.go $(XEN_GOPATH)$(GOXL_PKG_DIR)
-       $(INSTALL_DATA) types.gen.go $(XEN_GOPATH)$(GOXL_PKG_DIR)
-       $(INSTALL_DATA) helpers.gen.go $(XEN_GOPATH)$(GOXL_PKG_DIR)
 # NOTE: This target is called from libxl/Makefile:all.  Since that
 # target must finish before golang/Makefile is called, this is
 # currently safe.  It must not be called from anywhere else in the
@@ -43,23 +33,21 @@ idl-gen: $(GOXL_GEN_FILES)
 # in the LDFLAGS; and thus we need to add -L$(XEN_XENLIGHT) here
 # so that it can find the actual library.
 .PHONY: build
-build: package
+build: xenlight.go $(GOXL_GEN_FILES)
        CGO_CFLAGS="$(CFLAGS_libxenlight) $(CFLAGS_libxentoollog)" 
CGO_LDFLAGS="$(LDLIBS_libxenlight) $(LDLIBS_libxentoollog) -L$(XEN_XENLIGHT) 
-L$(XEN_LIBXENTOOLLOG)" $(GO) build -x
 .PHONY: install
 install: build
-       $(INSTALL_DATA) $(XEN_GOPATH)$(GOXL_PKG_DIR)xenlight.go 
-       $(INSTALL_DATA) $(XEN_GOPATH)$(GOXL_PKG_DIR)types.gen.go 
-       $(INSTALL_DATA) $(XEN_GOPATH)$(GOXL_PKG_DIR)helpers.gen.go 
+       $(INSTALL_DATA) xenlight.go $(DESTDIR)$(GOXL_INSTALL_DIR)
+       $(INSTALL_DATA) types.gen.go $(DESTDIR)$(GOXL_INSTALL_DIR)
+       $(INSTALL_DATA) helpers.gen.go $(DESTDIR)$(GOXL_INSTALL_DIR)
 .PHONY: uninstall
        rm -rf $(DESTDIR)$(GOXL_INSTALL_DIR)
 .PHONY: clean
-       $(RM) -r $(XEN_GOPATH)$(GOXL_PKG_DIR)
-       $(RM) $(XEN_GOPATH)/pkg/*/$(XEN_GOCODE_URL)/xenlight.a
 .PHONY: distclean
 distclean: clean
generated by git-patchbot for /home/xen/git/xen.git#staging



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.