Repository navigation
deps: merve: Don't include the vendored simdutf with --shared-simdutf #62621
commit-lint.yml
on: pull_request_target
lint-commit-message
23s
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
|