diff options
| author | Richard Levitte <levitte@openssl.org> | 2020-11-26 21:21:02 +0100 |
|---|---|---|
| committer | Richard Levitte <levitte@openssl.org> | 2021-02-23 19:08:39 +0100 |
| commit | 51d058cd9418508b48ec44dce6087ce730173832 (patch) | |
| tree | 5648f7703323be66c96d5a26e634b19b7e716a19 | |
| parent | 4f6aeabd65bf13795823f4a6f4a03c815e9d096f (diff) | |
appveyor.yml: clarify conditions for building the plain configuration
The "plain" configuration is only meant to be built for an '[extended tests]'
commit, or on the master branch. This isn't at all clear from the
scripts, and furthermore, we "skip" the plain configuration by running
the OpenSSL configuration script... and then nothing more.
Instead, we use AppVeyor configuration issues to specify when and when
not to build the "plain" configuration, and leave it to the scripts to
do the right thing using only $env:EXTENDED_TESTS.
Fixes #7958
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/13537)
| -rw-r--r-- | appveyor.yml | 47 |
1 files changed, 27 insertions, 20 deletions
diff --git a/appveyor.yml b/appveyor.yml index f0dfc7f5ba..20d81c1b12 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -12,9 +12,29 @@ environment: configuration: - shared - - plain - minimal +for: + - + only_commits: + message: /\[extended tests\]/ + configuration: + - shared + - plain + - minimal + environment: + EXTENDED_TESTS: yes + - + branches: + only: + - master + configuration: + - shared + - plain + - minimal + environment: + EXTENDED_TESTS: yes + before_build: - ps: >- Install-Module VSSetup -Scope CurrentUser @@ -43,12 +63,6 @@ before_build: - perl configdata.pm --dump - cd .. - ps: >- - If (-not $env:APPVEYOR_PULL_REQUEST_NUMBER` - -or (&git log -1 $env:APPVEYOR_PULL_REQUEST_HEAD_COMMIT | - Select-String "\[extended tests\]") ) { - $env:EXTENDED_TESTS="yes" - } - - ps: >- If ($env:BUILDONLY -or $env:MAKEVERBOSE) { $env:NMAKE="nmake" } Else { @@ -59,24 +73,17 @@ before_build: build_script: - cd _build - - ps: >- - If ($env:Configuration -Match "shared" -or $env:EXTENDED_TESTS) { - cmd /c "%NMAKE% build_all_generated 2>&1" - # Unfortunately, CL=/MP would not have parallelizing effect - cmd /c "%NMAKE% PERL=no-perl 2>&1" - } + - "%NMAKE% build_all_generated" + - "%NMAKE% PERL=no-perl" - cd .. test_script: - cd _build - ps: >- - If ($env:Configuration -Match "shared" -or $env:EXTENDED_TESTS) { - # Unfortunately, HARNESS_JOBS=4 would not have parallelizing effect - if ($env:EXTENDED_TESTS) { - cmd /c "%NMAKE% test HARNESS_VERBOSE_FAILURE=yes 2>&1" - } Else { - cmd /c "%NMAKE% test HARNESS_VERBOSE_FAILURE=yes TESTS=-test_fuzz 2>&1" - } + if ($env:EXTENDED_TESTS) { + cmd /c "%NMAKE% test VERBOSE_FAILURE=yes 2>&1" + } Else { + cmd /c "%NMAKE% test VERBOSE_FAILURE=yes TESTS=-test_fuzz 2>&1" } - ps: >- if ($env:EXTENDED_TESTS) { |