From 2a9348b00a28ff27a030e0bd10bda5859b562484 Mon Sep 17 00:00:00 2001
From: Bill Robinson <dseevr@users.noreply.github.com>
Date: Mon, 25 Sep 2017 17:59:40 -0700
Subject: [PATCH] Makefile,scripts/release.sh: revert deletion of
 TAR_FILENAME/TAR_FILE vars from Makefile and bring back `tar` target

This corrects an undesired change in logic introduced in https://github.com/contiv/netplugin/pull/961

Signed-off-by: Bill Robinson <dseevr@users.noreply.github.com>
---
 Makefile           | 17 ++++++++++++++++-
 scripts/release.sh | 15 ++++++++++++++-
 2 files changed, 30 insertions(+), 2 deletions(-)

diff --git a/Makefile b/Makefile
index fb8d3afc5..ee432d80e 100755
--- a/Makefile
+++ b/Makefile
@@ -15,6 +15,9 @@ NAME := netplugin
 VERSION_FILE := $(NAME)-version
 VERSION := `cat $(VERSION_FILE)`
 TAR_EXT := tar.bz2
+TAR_FILENAME := $(NAME)-$(VERSION).$(TAR_EXT)
+TAR_LOC := .
+TAR_FILE := $(TAR_LOC)/$(TAR_FILENAME)
 GO_MIN_VERSION := 1.7
 GO_MAX_VERSION := 1.8
 GO_VERSION := $(shell go version | cut -d' ' -f3 | sed 's/go//')
@@ -318,7 +321,19 @@ host-plugin-release:
 	@echo dev: (need docker login with user in contiv org)
 	docker plugin push ${CONTIV_V2PLUGIN_NAME}
 
+only-tar:
+
+tar: clean-tar
+	CONTIV_NODES=1 ${MAKE} build
+	@tar -jcf $(TAR_FILE) -C $(GOPATH)/src/github.com/contiv/netplugin/bin netplugin netmaster netctl contivk8s netcontiv -C $(GOPATH)/src/github.com/contiv/netplugin/scripts contrib/completion/bash/netctl -C $(GOPATH)/src/github.com/contiv/netplugin/scripts get-contiv-diags
+
+clean-tar:
+	@rm -f $(TAR_LOC)/*.$(TAR_EXT)
+	@rm -f ${VERSION_FILE}
+
 # GITHUB_USER and GITHUB_TOKEN are needed be set to run github-release
-release:
+release: tar
+	TAR_FILENAME=$(TAR_FILENAME) TAR_FILE=$(TAR_FILE) \
 	OLD_VERSION=${OLD_VERSION} BUILD_VERSION=${BUILD_VERSION} \
 	USE_RELEASE=${USE_RELEASE} scripts/release.sh
+	@make clean-tar
diff --git a/scripts/release.sh b/scripts/release.sh
index 4fdf2e33c..04b0c88db 100755
--- a/scripts/release.sh
+++ b/scripts/release.sh
@@ -12,6 +12,17 @@ if [ -z "$(which github-release)" ]; then
 	exit 1
 fi
 
+
+if [ -z "${TAR_FILENAME-}" ]; then
+	echo "TAR_FILENAME needs to be defined to make a release"
+	exit 1
+fi
+
+if [ ! -f "$TAR_FILE" ]; then
+	echo "TAR_FILE ($TAR_FILE) doesn't exist"
+	exit 1
+fi
+
 if [ -z "$BUILD_VERSION" ]; then
 	echo "A release requires BUILD_VERSION to be defined"
 	exit 1
@@ -43,4 +54,6 @@ else
 fi
 
 set -x
-( (github-release -v release $pre_release -r netplugin -t $BUILD_VERSION -d "**Changelog**<br/>$changelog")) || exit 1
+( (github-release -v release $pre_release -r netplugin -t $BUILD_VERSION -d "**Changelog**<br/>$changelog") \
+	&& (github-release -v upload -r netplugin -t $BUILD_VERSION -n $TAR_FILENAME -f $TAR_FILE \
+		|| github-release -v delete -r netplugin -t $BUILD_VERSION)) || exit 1