mirror of
git://git.yoctoproject.org/meta-virtualization.git
synced 2025-07-19 12:50:22 +02:00
5b92dace1d
Part of building cri-o is the generation of the 'conmon/config.h' file. The content of this file is dynamic in that it has buffer sizes and a socket path which will depend on constants that are set in other parts of the code. For example the socket path can be setup for Windows or for Linux. To generate 'conmon/config.h' a small GO application is built and run called crio-config. This isn't really suitable for a -native package but we do have to run crio-config on the build host so we don't want to cross compile it. We therefor use the native GO to build this. This change allows things to work when the build host arch and the target arch don't match. A small update to the Makefile avoids mixing build host arch and target arch GO packages. Finally, We drop the crio-config binary from the install as it is only used to create the conmon/config.h as part of the build. This is consistent with the Makefile's install rule which does not install this binary as it has no use on the target. Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com> Signed-off-by: Bruce Ashfield <bruce.ashfield@gmail.com>
44 lines
1.6 KiB
Diff
44 lines
1.6 KiB
Diff
From 3822e834d2dbd87a4dc8cdd36e7fe3b0e9020c4f Mon Sep 17 00:00:00 2001
|
|
From: Mark Asselstine <mark.asselstine@windriver.com>
|
|
Date: Tue, 9 Apr 2019 13:52:59 -0400
|
|
Subject: [PATCH] Makefile: skip install when generating the config.h
|
|
|
|
When running 'go build' "The -i flag installs the packages that are
|
|
dependencies of the target." The Makefile makes use of this since many
|
|
of the dependencies needed to complete this rule will be used to by
|
|
other rules in the Makefile, thus speeding the overall build time
|
|
(avoiding redoing work).
|
|
|
|
In our case the crio-config tool and its dependencies are not being
|
|
cross-compiled as they have to run locally to produce the
|
|
conmon/config.h file and thus installing them is not useful. In this
|
|
case there are no savings and can actually result in errors during the
|
|
build or if the build ARCH and target ARCH are the same, a potential
|
|
for host contamination.
|
|
|
|
So we drop the use of '-i'.
|
|
|
|
Upstream-Status: Inappropriate [cross compile issue]
|
|
|
|
Signed-off-by: Mark Asselstine <mark.asselstine@windriver.com>
|
|
---
|
|
Makefile | 2 +-
|
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
|
diff --git a/src/import/Makefile b/src/import/Makefile
|
|
index b9fa97f..8336a5b 100644
|
|
--- a/src/import/Makefile
|
|
+++ b/src/import/Makefile
|
|
@@ -109,7 +109,7 @@ release-note:
|
|
@$(GOPATH)/bin/release-tool -n $(release)
|
|
|
|
conmon/config.h: cmd/crio-config/config.go oci/oci.go
|
|
- $(GO) build -i $(LDFLAGS) -o bin/crio-config $(PROJECT)/cmd/crio-config
|
|
+ $(GO) build $(LDFLAGS) -o bin/crio-config $(PROJECT)/cmd/crio-config
|
|
( cd conmon && $(CURDIR)/bin/crio-config )
|
|
|
|
clean:
|
|
--
|
|
2.7.4
|
|
|