yetus-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From a.@apache.org
Subject [yetus] branch master updated: YETUS-886. maven_reorder_module_process is eating the directory stack (#59)
Date Tue, 21 May 2019 03:27:48 GMT
This is an automated email from the ASF dual-hosted git repository.

aw pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/yetus.git


The following commit(s) were added to refs/heads/master by this push:
     new cfdb402  YETUS-886. maven_reorder_module_process is eating the directory stack (#59)
cfdb402 is described below

commit cfdb40219484cc7ebb72dc5b47febbf96bea99f5
Author: Allen Wittenauer <aw@apache.org>
AuthorDate: Mon May 20 20:27:44 2019 -0700

    YETUS-886. maven_reorder_module_process is eating the directory stack (#59)
---
 precommit/src/main/shell/core.d/change-analysis.sh | 1 -
 precommit/src/main/shell/test-patch.d/maven.sh     | 5 ++---
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/precommit/src/main/shell/core.d/change-analysis.sh b/precommit/src/main/shell/core.d/change-analysis.sh
index 3c4f40e..d1ac42a 100755
--- a/precommit/src/main/shell/core.d/change-analysis.sh
+++ b/precommit/src/main/shell/core.d/change-analysis.sh
@@ -278,5 +278,4 @@ function find_changed_modules
     "${BUILDTOOL}_reorder_modules" "${repostatus}"
   fi
   popd >/dev/null || return 1
-
 }
diff --git a/precommit/src/main/shell/test-patch.d/maven.sh b/precommit/src/main/shell/test-patch.d/maven.sh
index 22ad6a4..8d022ec 100755
--- a/precommit/src/main/shell/test-patch.d/maven.sh
+++ b/precommit/src/main/shell/test-patch.d/maven.sh
@@ -694,17 +694,16 @@ function maven_reorder_module_process
     fi
   done
 
+  pushd "${BASEDIR}" >/dev/null || return 1
   if [[ "${BUILDMODE}" == patch ]] && [[ "${MAVEN_FOUND_ROOT_POM}" == true ]]; then
 
     echo ""
     echo "Testing root pom.xml file for version change"
     echo ""
-    fn=$(module_file_fragment "${CHANGED_UNION_MODULES}")
-    pushd "${BASEDIR}/${CHANGED_UNION_MODULES}" >/dev/null || return 1
 
     # shellcheck disable=SC2046
     echo_and_redirect \
-      "${PATCH_DIR}/maven-${repostatus}-version-${fn}.txt" \
+      "${PATCH_DIR}/maven-${repostatus}-version-log.txt" \
       $("${BUILDTOOL}_executor") \
         "-fae" \
         "org.apache.maven.plugins:maven-help-plugin:evaluate" \


Mime
View raw message