From 1773ae571103144e2c1bde03079f9a17c2aa2d53 Mon Sep 17 00:00:00 2001 From: Viktor Szakats Date: Wed, 20 Aug 2025 20:33:50 +0200 Subject: [PATCH] replace --ci with --buildinfo --- RELEASE-NOTES | 2 -- docs/runtests.md | 8 ++++---- tests/CMakeLists.txt | 2 +- tests/Makefile.am | 2 +- tests/globalconfig.pm | 4 ++-- tests/runtests.pl | 8 ++++---- 6 files changed, 12 insertions(+), 14 deletions(-) diff --git a/RELEASE-NOTES b/RELEASE-NOTES index 770daed1de..f688b8937f 100644 --- a/RELEASE-NOTES +++ b/RELEASE-NOTES @@ -142,7 +142,6 @@ This release includes the following bugfixes: o pytest: increase server KeepAliveTimeout [26] o pytest: relax error check on test_07_22 [16] o resolving: dns error tracing [196] - o runtests: add `--ci` option, show `Env:` only when non-empty [134] o runtests: assume `Time::HiRes`, drop Perl Win32 dependency [163] o schannel: add an error message for client cert not found [165] o schannel: assume `CERT_CHAIN_REVOCATION_CHECK_CHAIN` [114] @@ -376,7 +375,6 @@ References to bug reports and discussions on issues: [131] = https://curl.se/bug/?i=18187 [132] = https://curl.se/bug/?i=18160 [133] = https://curl.se/bug/?i=18201 - [134] = https://curl.se/bug/?i=18147 [135] = https://curl.se/bug/?i=18143 [136] = https://curl.se/bug/?i=18142 [137] = https://curl.se/bug/?i=18141 diff --git a/docs/runtests.md b/docs/runtests.md index 0742444c54..4a7ddf5c0b 100644 --- a/docs/runtests.md +++ b/docs/runtests.md @@ -96,15 +96,15 @@ Provide a path to a curl binary to talk to APIs (currently only CI test APIs). Display test results in automake style output (`PASS/FAIL: [number] [name]`). +## `--buildinfo` + +Dump `buildinfo.txt`. + ## `-c \` Provide a path to a custom curl binary to run the tests with. Default is the curl executable in the build tree. -## `--ci` - -Show extra information useful in for CI runs. - ## `-d` Enable protocol debug: have the servers display protocol output. If used in diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt index 0593b513ee..657d9fe09a 100644 --- a/tests/CMakeLists.txt +++ b/tests/CMakeLists.txt @@ -126,7 +126,7 @@ curl_add_runtests(test-am "-a -am") curl_add_runtests(test-full "-a -p -r") # ~flaky means that it ignores results of tests using the flaky keyword curl_add_runtests(test-nonflaky "-a -p ~flaky ~timing-dependent") -curl_add_runtests(test-ci "-a -p ~flaky ~timing-dependent -r --retry=5 -j20 --ci") +curl_add_runtests(test-ci "-a -p ~flaky ~timing-dependent -r --retry=5 -j20 --buildinfo") curl_add_runtests(test-torture "-a -t -j20") curl_add_runtests(test-event "-a -e") diff --git a/tests/Makefile.am b/tests/Makefile.am index 3894355fe9..639da33809 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -129,7 +129,7 @@ TEST_E = -a -e TEST_NF = -a -p ~flaky ~timing-dependent # special CI target derived from nonflaky with CI-specific flags -TEST_CI = $(TEST_NF) -r --retry=5 -j20 --ci +TEST_CI = $(TEST_NF) -r --retry=5 -j20 --buildinfo PYTEST = pytest endif diff --git a/tests/globalconfig.pm b/tests/globalconfig.pm index 9fa436d8bb..6ea605a219 100644 --- a/tests/globalconfig.pm +++ b/tests/globalconfig.pm @@ -46,7 +46,7 @@ BEGIN { $TUNITDIR $SRVDIR $listonly - $ci + $buildinfo $LOCKDIR $LOGDIR $memanalyze @@ -95,7 +95,7 @@ our $verbose; # 1 to show verbose test output our $torture; # 1 to enable torture testing our $proxy_address; # external HTTP proxy address our $listonly; # only list the tests -our $ci; # show extra info useful in CI runs +our $buildinfo; # dump buildinfo.txt our $run_duphandle; # run curl with --test-duphandle to verify handle duplication our $run_event_based; # run curl with --test-event to test the event API our $automakestyle; # use automake-like test status output format diff --git a/tests/runtests.pl b/tests/runtests.pl index 65b2482d6f..617145b7e5 100755 --- a/tests/runtests.pl +++ b/tests/runtests.pl @@ -2440,8 +2440,8 @@ while(@ARGV) { # lists the test case names only $listonly=1; } - elsif($ARGV[0] eq "--ci") { - $ci=1; + elsif($ARGV[0] eq "--buildinfo") { + $buildinfo=1; } elsif($ARGV[0] =~ /^-j(.*)/) { # parallel jobs @@ -2496,8 +2496,8 @@ Usage: runtests.pl [options] [test selection(s)] -a continue even if a test fails -ac path use this curl only to talk to APIs (currently only CI test APIs) -am automake style output PASS/FAIL: [number] [name] + --buildinfo dump buildinfo.txt -c path use this curl executable - --ci show extra info useful in for CI runs (e.g. buildinfo.txt dump) -d display server debug info -e, --test-event event-based execution --test-duphandle duplicate handles before use @@ -2685,7 +2685,7 @@ if(!$listonly) { ####################################################################### # Output information about the curl build # -if(!$listonly && $ci) { +if(!$listonly && $buildinfo) { if(open(my $fd, "<", "../buildinfo.txt")) { while(my $line = <$fd>) { chomp $line;