Bug 280673 - www/node20: fix build with libc++ 19
Summary: www/node20: fix build with libc++ 19
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Po-Chuan Hsieh
URL:
Keywords:
Depends on:
Blocks: 280562
  Show dependency treegraph
 
Reported: 2024-08-07 17:10 UTC by Dimitry Andric
Modified: 2024-08-15 12:08 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (sunpoet)


Attachments
www/node20: fix build with libc++ 19 (9.22 KB, patch)
2024-08-07 17:10 UTC, Dimitry Andric
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitry Andric freebsd_committer freebsd_triage 2024-08-07 17:10:24 UTC
As noted in the libc++ 19 release notes [1], std::char_traits<> is now
only provided for char, char8_t, char16_t, char32_t and wchar_t, and any
instantiation for other types will fail.

This causes www/node20 to fail to compile with clang 19 and libc++ 19,
resulting in errors similar to:

    /usr/include/c++/v1/string:820:42: error: implicit instantiation of undefined template 'std::char_traits<unsigned short>'
      820 |   static_assert(is_same<_CharT, typename traits_type::char_type>::value,
          |                                          ^
    ../deps/v8/src/inspector/string-16.h:114:28: note: in instantiation of template class 'std::basic_string<unsigned short>' requested here
      114 |   std::basic_string<UChar> m_impl;
          |                            ^
    /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared here
       23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
          |                             ^

Upstream v8 has fixed this in commit 182d9c05e78 [2], so add it as a
backported patch, until the next version of node is released.

[1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-removals
[2] https://chromium.googlesource.com/v8/v8.git/+/182d9c05e78
Comment 1 Dimitry Andric freebsd_committer freebsd_triage 2024-08-07 17:10:49 UTC
Created attachment 252593 [details]
www/node20: fix build with libc++ 19
Comment 2 commit-hook freebsd_committer freebsd_triage 2024-08-15 11:56:51 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=f832d50a9e7c59a3a0ce400cadb49b2e72241f6e

commit f832d50a9e7c59a3a0ce400cadb49b2e72241f6e
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-08-15 09:47:21 +0000
Commit:     Po-Chuan Hsieh <sunpoet@FreeBSD.org>
CommitDate: 2024-08-15 11:53:54 +0000

    www/node20: Fix build with libc++ 19

    PR:             280673
    Obtained from:  https://chromium.googlesource.com/v8/v8.git/+/182d9c05e78b1ddb1cb8242cd3628a7855a0336f

 www/node20/files/patch-libc++19 (new) | 185 ++++++++++++++++++++++++++++++++++
 1 file changed, 185 insertions(+)
Comment 3 Po-Chuan Hsieh freebsd_committer freebsd_triage 2024-08-15 12:08:45 UTC
Committed. Thanks!