Skip to content

tests: fix prechecks to call the bundle libtest tool#18099

Closed
vszakats wants to merge 5 commits intocurl:masterfrom
vszakats:precheck-bundle
Closed

tests: fix prechecks to call the bundle libtest tool#18099
vszakats wants to merge 5 commits intocurl:masterfrom
vszakats:precheck-bundle

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Jul 30, 2025

Some tests make a hard-coded call to the libtest binary in the precheck
step. With bundle builds the binary changed name and calling convention.
Before this patch these tests failed the pre-check and did not run for
the 5 affected tests: 518, 537, 678, 1517, 1960

Fixing, e.g.:

test 1517 SKIPPED: precheck command error

https://github.com/curl/curl/actions/runs/16611990422/job/46996698437?pr=18039#step:13:4832

It also fixes builds with a custom CURL_DIRSUFFIX set.

Follow-up to 2c27a67 #17590
Follow-up to 71cf0d1 #14772

@vszakats vszakats added the tests label Jul 30, 2025
@vszakats vszakats marked this pull request as draft July 30, 2025 11:29
@vszakats

This comment was marked as outdated.

@vszakats vszakats marked this pull request as ready for review July 30, 2025 12:14
@vszakats vszakats closed this in 801800a Jul 30, 2025
@vszakats vszakats deleted the precheck-bundle branch July 30, 2025 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

1 participant