cmake: replace internal option with a new testbins target - #20708
Closed
vszakats wants to merge 7 commits into
Closed
cmake: replace internal option with a new testbins target#20708vszakats wants to merge 7 commits into
testbins target#20708vszakats wants to merge 7 commits into
Conversation
vszakats
marked this pull request as draft
February 24, 2026 11:42
Member
Author
|
it seems the CI infra is struggling today. |
Member
Author
|
Also the Azure Ubuntu mirrors did not recover yet from yesterday. |
vszakats
marked this pull request as ready for review
February 24, 2026 13:29
vszakats
force-pushed
the
cm-cert-and-tests
branch
from
February 24, 2026 15:26
798d0a6 to
519378d
Compare
vszakats
added a commit
to vszakats/curl
that referenced
this pull request
Feb 26, 2026
- run tests-clang-tidy when building testbins. - drop redundant build-certs dependency for test targets. Already there via testdeps. Follow-up to aae3612 curl#20708
vszakats
added a commit
to vszakats/curl
that referenced
this pull request
Feb 28, 2026
outcast36
pushed a commit
to greearb/curl
that referenced
this pull request
Jun 3, 2026
To allow building test binaries without test certs, replacing `_CURL_SKIP_BUILD_CERTS` internal option with a build target that is similar to `testdeps`, but without building the test certificates. To make building test binaries a little bit faster, with less noisy output, and without having to reconfigure the build. Closes curl#20708
outcast36
pushed a commit
to greearb/curl
that referenced
this pull request
Jun 3, 2026
- run tests-clang-tidy when building testbins. - drop redundant build-certs dependency for test targets. Already present via testdeps. Follow-up to aae3612 curl#20708 Closes curl#20727
outcast36
pushed a commit
to greearb/curl
that referenced
this pull request
Jun 3, 2026
To make it easy to type. The internal option is used in CI. Follow-up to aae3612 curl#20708 Closes curl#20768
vszakats
added a commit
to curl/curl-for-win
that referenced
this pull request
Jun 26, 2026
The difference is that `tt` does not build the test certificates. To save a little build time. Refs: curl/curl@1b35c9e curl/curl#20768 curl/curl@aae3612 curl/curl#20708
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
To allow building test binaries without test certs, replacing
_CURL_SKIP_BUILD_CERTSinternal option with a build target that issimilar to
testdeps, but without building the test certificates.To make building test binaries a little bit faster, with less noisy
output, and without having to reconfigure the build.
testbinstarget tott, restore internal option #20768