Fix release file detection 58/62958/2
authorAric Gardner <agardner@linuxfoundation.org>
Mon, 27 Jan 2020 19:49:59 +0000 (14:49 -0500)
committerLott, Christopher (cl778h) <cl778h@att.com>
Fri, 31 Jan 2020 20:22:24 +0000 (15:22 -0500)
commits with multiple parents were not being detected.
break long shell command across two lines

ISSUE-ID: RELENG-2685
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>
Change-Id: I660fabfba0497ab4c6825853c0fa117bacc93581
Signed-off-by: Lott, Christopher (cl778h) <cl778h@att.com>
releasenotes/notes/lf-release-jobs-911087ad53477868.yaml [new file with mode: 0644]
shell/release-job.sh

diff --git a/releasenotes/notes/lf-release-jobs-911087ad53477868.yaml b/releasenotes/notes/lf-release-jobs-911087ad53477868.yaml
new file mode 100644 (file)
index 0000000..6e07889
--- /dev/null
@@ -0,0 +1,4 @@
+---
+fixes:
+  - |
+    Fix release file detection on commit with multiple parents
index ee5ab70..afc19fc 100644 (file)
@@ -29,10 +29,12 @@ set_variables_common(){
     NEXUS_PATH="${SILO}/${JENKINS_HOSTNAME}/"
     # Verify if using release file or parameters
     if $USE_RELEASE_FILE ; then
-        release_files=$(git diff-tree -m --no-commit-id -r "$GIT_COMMIT" --name-only -- "releases/" ".releases/")
+        release_files=$(git diff-tree -m --no-commit-id -r "$GIT_COMMIT" "$GIT_COMMIT^1" \
+            --name-only -- "releases/" ".releases/")
         if (( $(grep -c . <<<"$release_files") > 1 )); then
           echo "INFO: RELEASE FILES ARE AS FOLLOWS: $release_files"
-          echo "ERROR: Committing multiple release files in the same commit OR rename/amend of existing files is not supported."
+          echo "ERROR: Adding multiple release files in the same commit"
+          echo "ERROR: OR rename/amend/delete of existing files is not supported."
           exit 1
         else
           release_file="$release_files"