This is an automated email from the ASF dual-hosted git repository. sijie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/bookkeeper.git
The following commit(s) were added to refs/heads/master by this push: new 7584b1b Fix flaky travis build on checking license header 7584b1b is described below commit 7584b1b4809e961c90b400912cfd08f7f45f3bfb Author: Sijie Guo <si...@apache.org> AuthorDate: Tue Feb 20 02:09:32 2018 -0800 Fix flaky travis build on checking license header Descriptions of the changes in this PR: *Problem* In some environments, `BK_VERSION` is computed as `Downloading`, which is not a valid bookkeeper version. It fails the `dev/check-binary-license` script. ``` [0K$ if [ "$TRAVIS_OS_NAME" == "linux" ]; then dev/check-binary-license ./bookkeeper-dist/server/target/bookkeeper-server-${BK_VERSION}-bin.tar.gz; fi tar: ./bookkeeper-dist/server/target/bookkeeper-server-Downloading: Cannot open: No such file or directory tar: Error is not recoverable: exiting now tar: ./bookkeeper-dist/server/target/bookkeeper-server-Downloading: Cannot open: No such file or directory tar: Error is not recoverable: exiting now tar: ./bookkeeper-dist/server/target/bookkeeper-server-Downloading: Cannot open: No such file or directory tar: Error is not recoverable: exiting now ``` *Solution* Remove this `BK_VERSION` setting and use "*" in the file path. Author: Sijie Guo <si...@apache.org> Reviewers: Ivan Kelly <iv...@apache.org> This closes #1183 from sijie/address_travis_yml --- .travis.yml | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/.travis.yml b/.travis.yml index c56c458..039999e 100644 --- a/.travis.yml +++ b/.travis.yml @@ -54,12 +54,11 @@ before_install: echo "Enable testing distributedlog modules if this pull request modifies files under directory `stream/distributedlog`." fi fi - export BK_VERSION=`mvn org.apache.maven.plugins:maven-help-plugin:2.1.1:evaluate -Dexpression=project.version | grep -Ev '(^\[|Download\w+:)' 2> /dev/null` script: - travis_retry mvn --batch-mode clean apache-rat:check compile spotbugs:check install -DskipTests - - if [ "$TRAVIS_OS_NAME" == "linux" ]; then dev/check-binary-license ./bookkeeper-dist/all/target/bookkeeper-all-${BK_VERSION}-bin.tar.gz; fi - - if [ "$TRAVIS_OS_NAME" == "linux" ]; then dev/check-binary-license ./bookkeeper-dist/server/target/bookkeeper-server-${BK_VERSION}-bin.tar.gz; fi + - if [ "$TRAVIS_OS_NAME" == "linux" ]; then dev/check-binary-license ./bookkeeper-dist/all/target/bookkeeper-all-*-bin.tar.gz; fi + - if [ "$TRAVIS_OS_NAME" == "linux" ]; then dev/check-binary-license ./bookkeeper-dist/server/target/bookkeeper-server-*-bin.tar.gz; fi - if [ "$DLOG_MODIFIED" == "true" ]; then cd stream/distributedlog && travis_wait 60 mvn --batch-mode clean package -Ddistributedlog; fi # Disabled the tests here. Since tests are running much slower on Travis than on Jenkins # - ./dev/ticktoc.sh "mvn --batch-mode clean package" -- To stop receiving notification emails like this one, please contact si...@apache.org.