commit 39fac65d3fb1cb2fe5212e1965e904cb953c9724
author Joey Armstrong <joey@opennetworking.org> Mon Mar 27 18:50:43 2023 -0400
committer Joey Armstrong <joey@opennetworking.org> Mon Mar 27 18:52:59 2023 -0400
tree 0c4187df7cda727f9745e6f9481a643f1b9e326b
parent 472bfbaa03864f108b7cf3f0938e94c99266459b

Env var is GERRIT_PROJECT not GIT_PROJECT

Change-Id: I31045a815e53286d2204ff6cad776413c262d14f

jjb/shell/github-release.sh

diff --git a/jjb/shell/github-release.sh b/jjb/shell/github-release.sh
index f2e55b4..e8d1c7c 100755
--- a/jjb/shell/github-release.sh
+++ b/jjb/shell/github-release.sh
@@ -264,11 +264,12 @@
     local ver
 
     banner
+
+    ## [TODO] move to get_version()
     ver="$(git tag -l --points-at HEAD)"
     declare -p ver
-
-    get_version 'ver'
-    declare -p ver
+    # get_version 'ver'
+    # declare -p ver
     
     # ------------------------------------------------------
     # match bare versions or v-prefixed golang style version
@@ -365,10 +366,10 @@
     local name
     if [[ -v __repo_name ]]; then
 	name="$__repo_name"
-    elif [[ ! -v GITHUB_PROJECT ]]; then
-	error "--repo-name or GITHUB_PROJECT= are required"
+    elif [[ ! -v GERRIT_PROJECT ]]; then
+	error "--repo-name or GERRIT_PROJECT= are required"
     else
-	name="${GITHUB_PROJECT}"
+	name="${GERRIT_PROJECT}"
     fi
 
     varname="$name"
@@ -531,11 +532,10 @@
 
     ## Flatten filesystem, should we recurse here to release subdirs ?
     #cp $(which ls) "$work"
-    # readarray -t arts < <(find "$artifact_glob" -type f)
     readarray -t artifacts < <(find "$work" -type f)
     func_echo "$(declare -p artifacts)"
 
-    # [[ ${#arts[@]} -eq 0 ]] && error "Artifact dir is empty, check for build failures in $artifact_glob"
+    [[ ${#artifacts[@]} -eq 0 ]] && error "Artifact dir is empty, check for build failures in $artifact_glob"
     
     # Copy artifacts into the release temp dir
     # shellcheck disable=SC2086