This is an automated email from the ASF dual-hosted git repository.

trohrmann pushed a commit to branch release-1.7
in repository https://gitbox.apache.org/repos/asf/flink.git


The following commit(s) were added to refs/heads/release-1.7 by this push:
     new 5c28c2c  [FLINK-10772][release] Fix create_binary_release.sh
5c28c2c is described below

commit 5c28c2c1f89535979ffc881eb4a3fd48fa0c6fa9
Author: Till Rohrmann <trohrm...@apache.org>
AuthorDate: Sun Nov 4 18:38:17 2018 +0100

    [FLINK-10772][release] Fix create_binary_release.sh
    
    Remove the unnecessary call to change_scala_version.sh and remove the
    -Dmaven.test.skip=true property. The latter is necessary because this
    property suppresses the compilation and packaging of test classes. It,
    however, does not suppress the resolution of test dependencies which
    will then fail to compile because test dependencies have not been built.
    
    This commit also removes the redundant call to build
    flink-shaded/hadoop/flink-shaded-hadoop2-uber which is a dependency
    of flink-dist anyway.
---
 tools/releasing/create_binary_release.sh | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/tools/releasing/create_binary_release.sh 
b/tools/releasing/create_binary_release.sh
index 8146127..f4d7c57 100755
--- a/tools/releasing/create_binary_release.sh
+++ b/tools/releasing/create_binary_release.sh
@@ -78,8 +78,7 @@ make_binary_release() {
   fi
 
   # enable release profile here (to check for the maven version)
-  tools/change-scala-version.sh ${SCALA_VERSION}
-  $MVN clean package $FLAGS -Prelease -pl 
flink-shaded-hadoop/flink-shaded-hadoop2-uber,flink-dist -am -Dgpg.skip 
-Dcheckstyle.skip=true -DskipTests -Dmaven.test.skip=true
+  $MVN clean package $FLAGS -Prelease -pl flink-dist -am -Dgpg.skip 
-Dcheckstyle.skip=true -DskipTests
 
   cd flink-dist/target/flink-*-bin/
   tar czf "${dir_name}.tgz" flink-*

Reply via email to