Revamp the way we package cap and psx packages.

It has been requested that we make the "libcap/cap" package into
a module that plays better with the golang ecosystem. I was holding
off until there was a golang version that contained the
runtime.AllThreadsSyscall() support, but that appears to not have
made it to 1.15, so I'm using a development build tag dependency
in otherwise static sources for the "libcap/cap" package.

My intention is that the canonical import paths for these packages
will be:

   "git.kernel.org/libs/libcap/cap"
   "git.kernel.org/libs/libcap/psx"

That being said, I may have to move them if I can't get some
proxy to resolve these paths to the right git repo of kernel.org.

The is work in the direction of addressing:

   https://bugzilla.kernel.org/show_bug.cgi?id=207567

Signed-off-by: Andrew G. Morgan <morgan@kernel.org>

1 files changed, 31 insertions, 28 deletions

diff --git a/go/Makefile b/go/Makefile
index 7269716..3bee774 100644
--- a/go/Makefile
+++ b/go/Makefile
@@ -1,12 +1,20 @@
-# Building the libcap/{cap.psx} Go packages. Note, we use symlinks to
-# construct a go friendly src tree.
+# Building the libcap/{cap.psx} Go packages, and examples.
+#
+# Note, we use symlinks to construct a GOPATH friendly src tree. The
+# packages themselves are intended to be (ultimately) found via proxy
+# as "git.kernel.org/lib/libcap/cap" and
+# "git.kernel.org/lib/libcap/psx". However, to validate their use on
+# these paths, we fake such a structure in the build tree with
+# symlinks.
 
 topdir=$(realpath ..)
 include $(topdir)/Make.Rules
 
 GOPATH=$(realpath .)
-PSXGOPACKAGE=pkg/$(GOOSARCH)/libcap/psx.a
-CAPGOPACKAGE=pkg/$(GOOSARCH)/libcap/cap.a
+IMPORTDIR=git.kernel.org/libs/libcap
+PKGDIR=pkg/$(GOOSARCH)/$(IMPORTDIR)
+PSXGOPACKAGE=$(PKGDIR)/psx.a
+CAPGOPACKAGE=$(PKGDIR)/cap.a
 
 DEPS=../libcap/libcap.a ../libcap/libpsx.a
 
@@ -18,29 +26,27 @@ $(DEPS):
 ../progs/capsh:
 	make -C ../progs capsh
 
-src/libcap/psx:
-	mkdir -p src/libcap
-	ln -s $(topdir)/psx src/libcap/
+src/$(IMPORTDIR)/psx:
+	mkdir -p src/$(IMPORTDIR)
+	ln -s $(topdir)/psx src/$(IMPORTDIR)
 
-src/libcap/cap:
-	mkdir -p src/libcap
-	ln -s $(topdir)/cap src/libcap/
+src/$(IMPORTDIR)/cap:
+	mkdir -p src/$(IMPORTDIR)
+	ln -s $(topdir)/cap src/$(IMPORTDIR)
 
 $(topdir)/libcap/cap_names.h: $(DEPS)
 	make -C $(topdir)/libcap all
 
-src/libcap/cap/names.go: $(topdir)/libcap/cap_names.h src/libcap/cap  mknames.go
+good-names.go: $(topdir)/libcap/cap_names.h src/$(IMPORTDIR)/cap  mknames.go
 	go run mknames.go --header=$< | gofmt > $@ || rm -f $@
+	diff -u ../cap/names.go $@
 
-src/libcap/cap/syscalls.go: ./syscalls.sh src/libcap/cap
-	./syscalls.sh src/libcap/cap
-
-$(PSXGOPACKAGE): src/libcap/psx src/libcap/psx/psx.go src/libcap/psx/psx_test.go $(DEPS)
+$(PSXGOPACKAGE): src/$(IMPORTDIR)/psx ../psx/*.go $(DEPS)
 	mkdir -p pkg
-	CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH="$(GOPATH)" go install libcap/psx
+	CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH="$(GOPATH)" go install $(IMPORTDIR)/psx
 
-$(CAPGOPACKAGE): src/libcap/cap/syscalls.go src/libcap/cap/names.go src/libcap/cap/cap.go src/libcap/cap/text.go $(PSXGOPACKAGE)
-	CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH=$(GOPATH) go install libcap/cap
+$(CAPGOPACKAGE): src/$(IMPORTDIR)/cap ../cap/*.go good-names.go $(PSXGOPACKAGE)
+	CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH=$(GOPATH) go install $(IMPORTDIR)/cap
 
 # Compiles something with this package to compare it to libcap. This
 # tests more when run under sudotest (see ../progs/quicktest.sh for that).
@@ -48,25 +54,25 @@ compare-cap: compare-cap.go $(CAPGOPACKAGE)
 	CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH=$(GOPATH) go build $<
 
 web: web.go $(CAPGOPACKAGE)
-	CGO_ENABLED="$(CGO_REQUIRED)" CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH=$(GOPATH) go build $<
+	CGO_ENABLED="$(CGO_REQUIRED)" CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH=$(GOPATH) go build $(GOBUILDTAG) $<
 ifeq ($(RAISE_GO_FILECAP),yes)
 	make -C ../progs setcap
 	sudo ../progs/setcap cap_setpcap,cap_net_bind_service=p web
-	@echo "NOTE: RAISED cap_net_bind_service ON web binary"
+	@echo "NOTE: RAISED cap_setpcap,cap_net_bind_service ON web binary"
 endif
 
 ok: ok.go
 	CGO_ENABLED=0 GOPATH=$(GOPATH) go build $<
 
 try-launching: try-launching.go $(CAPGOPACKAGE) ok
-	CGO_ENABLED="$(CGO_REQUIRED)" CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH=$(GOPATH) go build $<
+	CGO_ENABLED="$(CGO_REQUIRED)" CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH=$(GOPATH) go build $(GOBUILDTAG) $<
 ifeq ($(CGO_REQUIRED),0)
 	CGO_ENABLED="1" CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH=$(GOPATH) go build -o $@-cgo $<
 endif
 
 test: all ../progs/capsh
-	CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH="$(GOPATH)" go test libcap/psx
-	CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH=$(GOPATH) go test libcap/cap
+	CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH="$(GOPATH)" go test $(IMPORTDIR)/psx
+	CGO_LDFLAGS_ALLOW="$(CGO_LDFLAGS_ALLOW)" CGO_CFLAGS="$(CGO_CFLAGS)" CGO_LDFLAGS="$(CGO_LDFLAGS)" GOPATH=$(GOPATH) go test $(IMPORTDIR)/cap
 	LD_LIBRARY_PATH=../libcap ./compare-cap
 	./try-launching
 ifeq ($(CGO_REQUIRED),0)
@@ -88,10 +94,7 @@ install: all
 	install -m 0644 src/libcap/cap/* $(FAKEROOT)$(GOPKGDIR)/libcap/cap/
 
 clean:
-	GOPATH=$(GOPATH) go clean -x -i libcap/cap 2> /dev/null || exit 0
-	GOPATH=$(GOPATH) go clean -x -i libcap/psx 2> /dev/null || exit 0
-	rm -f *.o *.so *~ mknames web ok
+	rm -f *.o *.so *~ mknames web ok good-names.go
 	rm -f compare-cap try-launching try-launching-cgo
-	rm -f $(topdir)/cap/*~ $(topdir)/cap/names.go $(topdir)/cap/syscalls*.go
-	rm -f $(topdir)/psx/*~
+	rm -f $(topdir)/cap/*~ $(topdir)/psx/*~
 	rm -fr pkg src