Skip to content

deps: merve: Don't include the vendored simdutf with --shared-simdutf #62621

deps: merve: Don't include the vendored simdutf with --shared-simdutf

deps: merve: Don't include the vendored simdutf with --shared-simdutf #62621

Triggered via pull request November 11, 2026 13:28
@ziyao233ziyao233
opened #67410
Status Failure
Total duration 26s
Artifacts –

commit-lint.yml

on: pull_request_target
lint-commit-message
23s
lint-commit-message
Fit to window
Zoom out
Zoom in

Annotations

1 error
lint-commit-message
# dfa62df0b503475cf80a3355f01ca1ce5a6b8745 ok 1 assisted-by-is-trailer: no Assisted-by metadata ok 2 co-authored-by-is-trailer: no Co-authored-by metadata ok 3 fixes-url: skipping fixes-url # SKIP ok 4 line-after-title: blank line after title not ok 5 line-length: Line should be <= 72 columns. --- { found: 74, compare: '<=', wanted: 72, at: { line: 12, column: 72, body: [ '', 'Different versions of simdutf may provide different implementations for', 'the same name, for example, the vendored version provides a symbol for', 'simdutf::find(),', '', 'simdutf_warn_unused const char *find(const char *start, const char *end,', ' char character) noexcept;', '', 'while upstream simdutf 9.3.3 inlines it,', '', 'simdutf_warn_unused simdutf_really_inline simdutf_constexpr23 const char *', 'find(const char *start, const char *end, char character) noexcept {', ' /* An inline function */', ' ...', '}', '', "However, the vendored simdutf's headers are currently unconditionally", 'included for merve, regardless whether the shared simdutf is linked', 'against or not, leading to missing symbols at linking time like', '', 'mold: error: undefined symbol: simdutf::find(char const*, char const*, char)', '>>> referenced by merve.cpp', '>>> obj/deps/merve/libmerve.a(obj/deps/merve/merve.merve.o):(lexer::CJSLexer::addReexport(std::__1::basic_string_view<char, std::__1::char_traits<char>>, unsigned int))', '>>> referenced by merve.cpp', '>>> obj/deps/merve/libmerve.a(obj/deps/merve/merve.merve.o):(lexer::CJSLexer::addExport(std::__1::basic_string_view<char, std::__1::char_traits<char>>, unsigned int))', '', 'Do not include the vendored simdutf when compiling merve if shared', 'simdutf is used to ensure the headers match the linked library.', '', 'Signed-off-by: Yao Zi <me@ziyao.cc>' ] } } ... not ok 6 line-length: Line should be <= 72 columns. --- { found: 76, compare: '<=', wanted: 72, at: { line: 22, column: 72, body: [ '', 'Different versions of simdutf may provide different implementations for', 'the same name, for example, the vendored version provides a symbol for', 'simdutf::find(),', '', 'simdutf_warn_unused const char *find(const char *start, const char *end,', ' char character) noexcept;', '', 'while upstream simdutf 9.3.3 inlines it,', '', 'simdutf_warn_unused simdutf_really_inline simdutf_constexpr23 const char *', 'find(const char *start, const char *end, char character) noexcept {', ' /* An inline function */', ' ...', '}', '', "However, the vendored simdutf's headers are currently unconditionally", 'included for merve, regardless whether the shared simdutf is linked', 'against or not, leading to missing symbols at linking time like', '', 'mold: error: undefined symbol: simdutf::find(char const*, char const*, char)', '>>> referenced by merve.cpp', '>>> obj/deps/merve/libmerve.a(obj/deps/merve/merve.merve.o):(lexer::CJSLexer::addReexport(std::__1::basic_string_view<char, std::__1::char_traits<char>>, unsigned int))', '>>> referenced by merve.cpp', '>>> obj/deps/merve/libmerve.a(obj/deps/merve/merve.merve.o):(lexer::CJSLexer::addExport(std::__1::basic_string_view<char, std::__1::char_traits<char>>, unsigned int))', '', 'Do not include the vendored simdutf when compiling merve if shared', 'simdutf is used