Skip to content

appveyor: replace PowerShell with bash + parallel autotools#12572

Closed
vszakats wants to merge 4 commits intocurl:masterfrom
vszakats:avsh
Closed

appveyor: replace PowerShell with bash + parallel autotools#12572
vszakats wants to merge 4 commits intocurl:masterfrom
vszakats:avsh

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Dec 21, 2023

PowerShell works (after a steep development curve), but one property of
it stuck and kept causing unresolvable usability issues: With
$ErrorActionPreference=Stop, it does abort on failures, but shows only
the first line of the error message. In Continue mode, it shows the
full error message, but doesn't stop on all errors. Another issue is
PowerShell considering any stderr output as if the command failed (this
has been improved in 7.2 (2021-Nov), but fixed versions aren't running
in CI and will not be for a long time in all test images.)

Thus, we're going with bash.

Also:

  • use -j2 with autotools tests, making them finish 5-15 minutes per
    job faster.
  • omit POSIX_PATH_PREFIX.
  • use WINDIR.
  • prefer forward slashes.

Follow-up to 75078a4 #11999
Ref: #12444

Fixes #12560
Closes #12572

@vszakats vszakats added Windows Windows-specific CI Continuous Integration labels Dec 21, 2023
@vszakats vszakats marked this pull request as draft December 21, 2023 04:17
@vszakats vszakats changed the title appveyor: replace PowerShell with bash appveyor: replace PowerShell with bash + parallel autotools Dec 21, 2023
@vszakats vszakats marked this pull request as ready for review December 21, 2023 22:25
@vszakats vszakats closed this in 2d4d0c1 Dec 21, 2023
@vszakats vszakats deleted the avsh branch December 21, 2023 22:54
vszakats added a commit to vszakats/curl that referenced this pull request Dec 22, 2023
@vszakats vszakats mentioned this pull request Dec 22, 2023
vszakats added a commit that referenced this pull request Dec 23, 2023
- replace two remaining backslashes with forward slashes.
- tidy up the way we form and pass `TFLAGS`.

Follow-up to 2d4d0c1 #12572

Closes #12582
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration Windows Windows-specific

Development

Successfully merging this pull request may close these issues.

AppVeyor build failures do not always show in log output

2 participants