[PATCH 0/4] Fix tests with missing iconv(1) executable
- From
Patrick Steinhardt <ps@pks.im>
- Date
- Feb 9, 2026, 12:42 UTC
- Message-ID
- <20260209-b4-pks-ci-msvc-iconv-fixes-v1-0-1e3167cd8828@pks.im>
Hi,
I recently noticed that th MSVC-based tests in GitLab CI started to fail. The root cause is that the iconv(1) executable cannot be found on this platform anymore. This isn't entirely surprising: we depend on the Git for Windows environment to provide necessary shell tools, and that environment of course is not a fully fledged MSYS2 installation.
In any case, this patch series fixes those issues by building on top of the ICONV prerequisite. If the prereq isn't found, then we also don't assume that the iconv(1) executable exists.
An alternative strategy would be to introduce a new ICONV_EXECUTABLE prereq. But given that Git doesn't perform any kind of reencoding itself in case the ICONV support isn't built into it I found it to not be worth the additional hassle.
In any case, this patch series causes the MSVC jobs to pass again on GitLab CI.
Thanks!
Patrick
---
Patrick Steinhardt (4):
t4xxx: don't use iconv(1) without ICONV prereq
t4205: improve handling of ICONV prerequisite
t5550: add ICONV prereq to tests that use "$HTTPD_URL/error"
t6006: don't use iconv(1) without ICONV prereqt/t4041-diff-submodule-option.sh | 8 +++-- t/t4059-diff-submodule-not-initialized.sh | 8 +++-- t/t4060-diff-submodule-option-diff-format.sh | 8 +++-- t/t4205-log-pretty-formats.sh | 50 ++++++++++++++++------------ t/t5550-http-fetch-dumb.sh | 20 +++++------ t/t6006-rev-list-format.sh | 29 +++++++++++----- 6 files changed, 77 insertions(+), 46 deletions(-)
--- base-commit: 3e0db84c88c57e70ac8be8c196dfa92c5d656fbc change-id: 20260209-b4-pks-ci-msvc-iconv-fixes-13de4801643f