Builder curl-ares-solaris10-i386 Build #3629

Results:

Failed runtest

SourceStamp:

BuildSlave:

Reason:

The SingleBranchScheduler scheduler named 'schedule-curl-ares-solaris10-i386' triggered this build

Steps and Logfiles:

1. git update ( 11 secs )
   1. stdio
2. Runtest './tests/testcurl.pl --nogitpull ...' failed ( 3 hrs, 9 mins, 47 secs )
   1. stdio
   2. resultlog
3. Mail result 'cat resultlog ...' ( 3 secs )
   1. stdio

Build Properties:

Forced Build Properties:

Responsible Users:

1. Daniel Stenberg

   danielohnoyoudont@haxx.se
2. David Zhuang

   dzhuangohnoyoudont@roblox.com
3. Viktor Szakats

   commitohnoyoudont@vsz.me

Timing:

All Changes:

:

1. Change #243178

   Category	curl
   Changed by	Daniel Stenberg <danielohnoyoudont@haxx.se>
   Changed at	Mon 18 Aug 2025 09:57:45
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	e4406860346b546cda6bcacd30d5db07c8efce0b

   Comments

   ftp: use 'conn' instead of 'data->conn'
   In the ftp_state_use_port function, as we already use that local
   variable for all other conn accesses.
   
   Closes #18304

   Changed files

   • lib/ftp.c

2. Change #243179

   Category	curl
   Changed by	Daniel Stenberg <danielohnoyoudont@haxx.se>
   Changed at	Mon 18 Aug 2025 09:58:31
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	294ebba56588754f226c8ab49e85dd457b5a1df4

   Comments

   bufq: simplify condition
   'result' is always CURLE_AGAIN here
   
   Pointed out by CodeSonar
   
   Closes #18305

   Changed files

   • lib/bufq.c

3. Change #243184

   Category	curl
   Changed by	Viktor Szakats <commitohnoyoudont@vsz.me>
   Changed at	Mon 18 Aug 2025 11:08:53
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	a19bd4321030c6ea18e3bbec411dfd49961a1f71

   Comments

   cmake: honor `CMAKE_C_FLAGS` in test 1119 and 1167
   `CMAKE_C_FLAGS` is not set by curl, but may contain custom options
   required for a successful compiler run, when invoked by these tests.
   
   One such case is when configuring Visual Studio or clang-cl via compiler
   options, instead of envs.
   
   Cherry-picked from #18301
   Closes #18307

   Changed files

   • tests/CMakeLists.txt

4. Change #243188

   Category	curl
   Changed by	Viktor Szakats <commitohnoyoudont@vsz.me>
   Changed at	Mon 18 Aug 2025 11:19:50
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	11bb681ce8f3c98ce17d1b07629f4b679af01ed3

   Comments

   GHA/windows: tidy up running find in two build steps
   Cherry-picked from #18301

   Changed files

   • .github/workflows/windows.yml

5. Change #243195

   Category	curl
   Changed by	Daniel Stenberg <danielohnoyoudont@haxx.se>
   Changed at	Mon 18 Aug 2025 13:03:51
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	fc4ae23cc22a12d283a291c5986270fef88b9120

   Comments

   asyn-thrdd: fix Curl_async_pollset without socketpair
   - remove a superfluous if()
   - edit the #ifdef to only do the timeout checks when socketpair support
     is missing, as that code flow could not be reached otherways anyway
   
   CodeSonar pointed out possible issue that led me to see this.
   
   Closes #18306

   Changed files

   • lib/asyn-thrdd.c

6. Change #243197

   Category	curl
   Changed by	David Zhuang <dzhuangohnoyoudont@roblox.com>
   Changed at	Mon 18 Aug 2025 13:25:52
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	7dafe10db2f0b104317e6640670b4cd973a4e3da

   Comments

   ngtcp2: use custom mem funcs
   Pass curl's memory functions to the nghttp3 and ngtcp2 functions that
   allow them. This allows custom memory functions passed by the curl user
   to be used in nghttp3 and ngtcp2.
   
   Closes #18196

   Changed files

   • lib/vquic/curl_ngtcp2.c
   • lib/vquic/curl_osslq.c
   • lib/vquic/vquic.c
   • lib/vquic/vquic_int.h

7. Change #243200

   Category	curl
   Changed by	Viktor Szakats <commitohnoyoudont@vsz.me>
   Changed at	Mon 18 Aug 2025 13:56:35
   Repository	https://github.com/curl/curl.git
   Project	curl
   Branch	master
   Revision	a477789c4344d8231265d739f37787d026086333

   Comments

   GHA/windows: switch from MSBuild to Ninja for MSVC jobs running tests
   This patch fixes flakiness caused by MSBuild scanning the runtests.pl
   output for regex patterns. When finding a hit, it returns an error code
   to cmake, making the build test CI step fail. This happens rarely after
   an earlier mitigation tweaking outputs, but, as expected, it did not
   resolve it completely.
   
   MSBuild doesn't have an option to disable this behavior. To fix, this
   patch migrates the two affected jobs from MSBuild to Ninja. To align
   with existing multi-config logic, it uses the `Ninja Multi-Config`
   generator, which hasn't been tested before in CI.
   
   Switching to Ninja was not trivial. Visual Studio to this day relies on
   an MS-DOS batch file stored at an unstable location (containing spaces
   and parenthesis), to initialize its environment. Without this env,
   `cl.exe` is unable to find its own components. GHA does not initialize
   it (even if it did, it could only default to a single specific target).
   CMake helps with this when using a Visual Studio generator, but doesn't
   when using Ninja. (On local machines the VS installer adds a couple
   of Start menu items for launching pre-configured command prompts.)
   
   Ref: https://learn.microsoft.com/cpp/build/building-on-the-command-line
   
   The MS-DOS batches don't integrate well with CI envs and even less so
   with shell scripts. To avoid it, this patch uses manual configuration.
   Also without using environment variables, to make it easy to use and
   easy to debug and trace in logs. Configuring Visual Studio is relatively
   stable across releases and hasn't changed a whole lot in the last 2
   decades, but still may need more maintenance compared to llvm, or pretty
   much any other toolchain out there. On the upside, it allows to manually
   select compiler version, SDK version, cross-combinations, and allows
   choosing clang-cl. The configuration aims to find the latest of these
   automatically.
   
   Some traps that had to be avoided:
   - need to switch to MS-DOS short names to avoid spaces in the VS
     component paths.
   - need to switch to forward slashes to avoid confusing downstream tools
     with backslashes.
   - need to pass either MSYS2 for Windows-style path depending on setting.
   - need to use a trick to retrieve the oddly named `ProgramFiles(x86)`
     Windows env from shell script.
   - need to match VS version (2022) and edition (Enterprise), found on GHA
     runners.
   - need to pass the CMake generator via env so that the space in the name
     doesn't trip the shell when passed via a variable.
   - trash and unexpected dirs when detecting SDK/toolchain versions.
   - need to pass `-external:W0` to the C compiler to avoid MSVC warning:
     `D9007: '/external:I' requires '/external:W'; option ignored`
   - using cmake options only, to make it run without relying on envs and
     work out-of-the-box when running subsequent cmake sessions.
   - some others discovered while making work clang-cl locally in
     cross-builds.
   
   Ninja also improves performance in most cases (though wasn't a goal
   here). After this patch configure is significantly faster (1.5-2x),
   builds are a tiny bit faster, except examples which was twice as fast
   with MSBuild. Disk space use is 10% lower.
   
   MSBuild builds remain tested in AppVeyor CI and the UWP job.
   
   Before: https://github.com/curl/curl/actions/runs/17025737223/job/48260856051
   After: https://github.com/curl/curl/actions/runs/17027981486/job/48266133301
   
   Fixes:
   ```
     === Start of file stderr1635
        % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                       Dload  Upload   Total   Spent    Left  Speed
   
        0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
      100     4  100     4    0     0    449      0 --:--:-- --:--:-- --:--:--   500
   curl : (22) The requested URL returned error : 429 [D:\a\curl\curl\bld\tests\test-ci.vcxproj]
   CUSTOMBUILD : warning : Problem : HTTP error. Will retry in 1 second. 1 retry left. [D:\a\curl\curl\bld\tests\test-ci.vcxproj]
   [...]
   C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(254,5): error MSB8066: Custom build for 'D:\a\curl\curl\bld\CMakeFiles\621f80ddbb0fa48179f056ca77842ff0\test-ci.rule;D:\a\curl\curl\tests\CMakeLists.txt' exited with code -1. [D:\a\curl\curl\bld\tests\test-ci.vcxproj]
   Error: Process completed with exit code 1.
   ```
   Ref: https://github.com/curl/curl/actions/runs/16966304797/job/48091058271?pr=18287#step:13:3471
   
   Bug: https://github.com/curl/curl/discussions/14854#discussioncomment-14104166
   Ref: a19bd4321030c6ea18e3bbec411dfd49961a1f71 #18307
   Follow-up to 9463769f2e2dba9eeff554a88e5df5195d2c774b #16583
   
   Closes #18301

   Changed files

   • .github/workflows/windows.yml