libtests: make test 1503,1504,1505 use the 1502 binary#17591
Closed
libtests: make test 1503,1504,1505 use the 1502 binary#17591
Conversation
Adjust the differences at runtime instead of build-time, to avoid extra buillds.
vszakats
reviewed
Jun 11, 2025
bagder
force-pushed
the
bagder/lib1500-runtime
branch
from
25c0c2c to
efc93f6
Compare
make libtest/first.c use that to set the 'testnum' global variable Make lib1502.c use it
vszakats
added a commit
that referenced
this pull request
Jun 11, 2025
After this patch there is no more double/multiple compile of the same libtest source under a different libtest ID. Each libtest is compiled once, and changing behavior at runtime based on test ID. - drop recently added physical clones for two prevously multi-compiled tests: - merge lib587 into lib554 again, branch at runtime. - merge lib645 into lib643 again, branch at runtime. - replace existing dynamic branching to use `testnum` instead of a manually rolled `testno` based on an extra command-line argument. lib1571, lib1576. - mk-bundle.pl: stop defining `LIB*` macros. No longer used. - libtests: drop all `LIB*_C` guards. - Make these tests branch at runtime, stop building copies: - lib585, based on lib500 - lib565, based on lib510 - lib529, based on lib525 - lib527, lib532, based on lib526 - lib545, based on lib544 - lib548, based on lib547 - lib696, based on lib556 - lib584, based on lib589 - lib1539, based on lib1514 - lib1543, based on lib1518 - lib1917, based on lib1916 - lib1946, based on lib1940 - lib671, 672, 673, based on lib670 Follow-up to 02dd471 #17591 Follow-up to 6897aeb #17468 Closes #17598
denandz
pushed a commit
to denandz/curl
that referenced
this pull request
Jun 21, 2025
Adjust the differences at runtime instead of build-time, to avoid extra buillds. Set the `CURL_TESTNUM` env variable to pass test numbers to tests. Make libtest/first.c use that env variable to set the `testnum` global variable to allow tests to differ based on which test that runs it. Closes curl#17591
denandz
pushed a commit
to denandz/curl
that referenced
this pull request
Jun 21, 2025
After this patch there is no more double/multiple compile of the same libtest source under a different libtest ID. Each libtest is compiled once, and changing behavior at runtime based on test ID. - drop recently added physical clones for two prevously multi-compiled tests: - merge lib587 into lib554 again, branch at runtime. - merge lib645 into lib643 again, branch at runtime. - replace existing dynamic branching to use `testnum` instead of a manually rolled `testno` based on an extra command-line argument. lib1571, lib1576. - mk-bundle.pl: stop defining `LIB*` macros. No longer used. - libtests: drop all `LIB*_C` guards. - Make these tests branch at runtime, stop building copies: - lib585, based on lib500 - lib565, based on lib510 - lib529, based on lib525 - lib527, lib532, based on lib526 - lib545, based on lib544 - lib548, based on lib547 - lib696, based on lib556 - lib584, based on lib589 - lib1539, based on lib1514 - lib1543, based on lib1518 - lib1917, based on lib1916 - lib1946, based on lib1940 - lib671, 672, 673, based on lib670 Follow-up to 02dd471 curl#17591 Follow-up to 6897aeb curl#17468 Closes curl#17598
vszakats
added a commit
to vszakats/curl
that referenced
this pull request
Aug 1, 2025
Follow-up to 02dd471 curl#17591 Follow-up to d3594be curl#17136
Closed
vszakats
added a commit
that referenced
this pull request
Aug 1, 2025
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
2 participants
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.
Adjust the differences at runtime instead of build-time, to avoid extra builds.
Set the
CURL_TESTNUMenv variable to pass test numbers to tests.Make libtest/first.c use that env variable to set the
testnumglobal variable to allow tests to differ based on which test that runs it.