]> git.djapps.eu Git - pkg/ggml/sources/llama.cpp/commit
arg: try fixing test-args-parser randomly fails (#24826)
authorXuan-Son Nguyen <redacted>
Sat, 20 Jun 2026 17:45:27 +0000 (19:45 +0200)
committerGitHub <redacted>
Sat, 20 Jun 2026 17:45:27 +0000 (19:45 +0200)
commit75f460ac289e61eb3c2bb63c9487794a1ed514d1
tree78dfedaca2e1f6798116015f6d8058b7d0ec259f
parent8452824611be321246f33339727f60a90c02c277
arg: try fixing test-args-parser randomly fails (#24826)

* arg: try fixing test-args-parser randomly fails

* return ref

* try triggering the workflow

* exception wrapper

* wip

* test

* test 2

* arg: guard win32 utf8 argv override

make_utf8_argv rebuilds argv from GetCommandLineW to fix utf8 handling of
non ascii arguments on windows. the override runs unconditionally inside
common_params_parse, so it also clobbers a programmatic argv passed by a
caller. test-arg-parser builds a synthetic argv but then sees the real
process command line instead, the model argument is never parsed, and the
assert that expects success aborts via fastfail (0xC0000409). this shows up
as a random failure in the openvino windows workflow.

only override argv when its length matches the caller argc, so the utf8
repair still applies to real binaries while a programmatic argv stays intact.

---------

Co-authored-by: Pascal <redacted>
common/arg.cpp
tests/test-arg-parser.cpp