From e41a8afa7b6fa5475e3b58083eec79789acef1b8 Mon Sep 17 00:00:00 2001
From: Michael Hudson-Doyle <michael.hudson@linaro.org>
Date: Wed, 8 Oct 2014 15:31:43 +1300
Subject: [PATCH] finally, use code generation for INITSHA1 & INITPATH too

Signed-off-by: Michael Hudson-Doyle <michael.hudson@linaro.org>
Upstream-commit: c7c620dd9fce0e7b437ee185c7a58341f3b8e3aa
Component: engine
---
 components/engine/.gitignore                    |  1 +
 components/engine/hack/make.sh                  |  2 +-
 components/engine/hack/make/dynbinary           | 11 ++++++++++-
 components/engine/hack/make/dyntest-integration | 12 +++++++++---
 components/engine/hack/make/dyntest-unit        | 12 +++++++++---
 5 files changed, 30 insertions(+), 8 deletions(-)

diff --git a/components/engine/.gitignore b/components/engine/.gitignore
index d7dae009d6..21df574df6 100644
--- a/components/engine/.gitignore
+++ b/components/engine/.gitignore
@@ -29,3 +29,4 @@ docs/VERSION
 docs/GITCOMMIT
 dockerversion/static.go
 dockerversion/details.go
+dockerversion/init.go
diff --git a/components/engine/hack/make.sh b/components/engine/hack/make.sh
index eda96b5e26..a841c13424 100755
--- a/components/engine/hack/make.sh
+++ b/components/engine/hack/make.sh
@@ -94,7 +94,7 @@ if [ -z "$DOCKER_CLIENTONLY" ]; then
 	DOCKER_BUILDTAGS+=" daemon"
 fi
 
-rm -f dockerversion/static.go dockerversion/details.go
+rm -f dockerversion/static.go dockerversion/details.go dockerversion/init.go
 cat > dockerversion/details.go <<EOF
 // AUTOGENERATED FILE; see hack/make.sh
 package dockerversion
diff --git a/components/engine/hack/make/dynbinary b/components/engine/hack/make/dynbinary
index bcb4471498..171dd16ba3 100644
--- a/components/engine/hack/make/dynbinary
+++ b/components/engine/hack/make/dynbinary
@@ -47,7 +47,16 @@ fi
 # exported so that "dyntest" can easily access it later without recalculating it
 
 (
-	export LDFLAGS_STATIC_DOCKER="-X $DOCKER_PKG/dockerversion.INITSHA1 \"$DOCKER_INITSHA1\" -X $DOCKER_PKG/dockerversion.INITPATH \"$DOCKER_INITPATH\""
+	export LDFLAGS_STATIC_DOCKER=""
+	cat > dockerversion/init.go <<EOF
+// AUTOGENERATED FILE; see hack/make/dynbinary, hack/make/dyntest-integration and dyntest-unit
+package dockerversion
+
+func init() {
+	INITSHA1 = "$DOCKER_INITSHA1"
+	INITPATH = "$DOCKER_INITPATH"
+}
+EOF
 	export IAMSTATIC=false
 	export BUILDFLAGS=( "${BUILDFLAGS[@]/netgo /}" ) # disable netgo, since we don't need it for a dynamic binary
 	source "$(dirname "$BASH_SOURCE")/binary"
diff --git a/components/engine/hack/make/dyntest-integration b/components/engine/hack/make/dyntest-integration
index 1cc7349aba..5015c10969 100644
--- a/components/engine/hack/make/dyntest-integration
+++ b/components/engine/hack/make/dyntest-integration
@@ -11,8 +11,14 @@ fi
 
 (
 	export TEST_DOCKERINIT_PATH="$INIT"
-	export LDFLAGS_STATIC_DOCKER="
-		-X $DOCKER_PKG/dockerversion.INITSHA1 \"$DOCKER_INITSHA1\"
-	"
+	export LDFLAGS_STATIC_DOCKER=""
+	cat > dockerversion/init.go <<EOF
+// AUTOGENERATED FILE; see hack/make/dynbinary, hack/make/dyntest-integration and dyntest-unit
+package dockerversion
+
+func init() {
+	INITSHA1 = "$DOCKER_INITSHA1"
+}
+EOF
 	source "$(dirname "$BASH_SOURCE")/test-integration"
 )
diff --git a/components/engine/hack/make/dyntest-unit b/components/engine/hack/make/dyntest-unit
index cffef98512..32cd8136bc 100644
--- a/components/engine/hack/make/dyntest-unit
+++ b/components/engine/hack/make/dyntest-unit
@@ -11,8 +11,14 @@ fi
 
 (
 	export TEST_DOCKERINIT_PATH="$INIT"
-	export LDFLAGS_STATIC_DOCKER="
-		-X $DOCKER_PKG/dockerversion.INITSHA1 \"$DOCKER_INITSHA1\"
-	"
+	export LDFLAGS_STATIC_DOCKER=""
+	cat > dockerversion/init.go <<EOF
+// AUTOGENERATED FILE; see hack/make/dynbinary, hack/make/dyntest-integration and dyntest-unit
+package dockerversion
+
+func init() {
+	INITSHA1 = "$DOCKER_INITSHA1"
+}
+EOF
 	source "$(dirname "$BASH_SOURCE")/test-unit"
 )