Correct travis.yml to only build extended tests when explicitely asked to
authorRichard Levitte <levitte@openssl.org>
Wed, 12 Apr 2017 16:57:02 +0000 (18:57 +0200)
committerRichard Levitte <levitte@openssl.org>
Wed, 12 Apr 2017 17:28:28 +0000 (19:28 +0200)
The check for this was done by checking if $TRAVIS_EVENT_TYPE is
"pull_request".  The trouble is that when new data is pushed to an
already existing pull request, the event type is "push".

Better then to go with another documented variable, $TRAVIS_PULL_REQUEST,
which is "false" in non-PR builds.

Ref: https://docs.travis-ci.com/user/environment-variables/#Default-Environment-Variables

Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3205)

.travis.yml

index 24214e9bee05acec54bdc5eba31a6a3cc18013b4..cb08e40d42e63414ed78aabb36d76fa66d09fda0 100644 (file)
@@ -146,7 +146,7 @@ matrix:
 
 before_script:
     - env
-    - if [ "$TRAVIS_EVENT_TYPE" == "pull_request" -a -n "$EXTENDED_TEST" ]; then
+    - if [ "$TRAVIS_PULL_REQUEST" != "false" -a -n "$EXTENDED_TEST" ]; then
           (git log -1 $TRAVIS_COMMIT_RANGE | grep '\[extended tests\]' > /dev/null) || exit 0;
       fi
     - if [ -n "$DESTDIR" ]; then