Bug 280563 - devel/binutils: fix build with libc++ 19
Summary: devel/binutils: 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: Dimitry Andric
URL:
Keywords:
Depends on:
Blocks: 280562
  Show dependency treegraph
 
Reported: 2024-08-01 18:36 UTC by Dimitry Andric
Modified: 2024-08-02 08:04 UTC (History)
0 users

See Also:
cy: maintainer-feedback+


Attachments
devel/binutils: fix build with libc++ 19 (5.38 KB, patch)
2024-08-01 18:37 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-01 18:36:54 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 devel/binutils 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,
          |                                          ^
    ./stringpool.h:225:35: note: in instantiation of template class 'std::basic_string<unsigned short>' requested here
      225 |   { return this->add_with_length(s.data(), s.size(), copy, pkey); }
          |                                   ^
    stringpool.cc:530:7: note: in instantiation of member function 'gold::Stringpool_template<unsigned short>::add' requested here
      530 | class Stringpool_template<uint16_t>;
          |       ^
    /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared here
       23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
          |                             ^

Upstream binutils has fixed this in commit 5e9091dab88 [2], so import it
as a separate patch, until the next version of binutils is released.

[1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-removals
[2] https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=5e9091dab88
Comment 1 Dimitry Andric freebsd_committer freebsd_triage 2024-08-01 18:37:19 UTC
Created attachment 252436 [details]
devel/binutils: fix build with libc++ 19
Comment 2 Cy Schubert freebsd_committer freebsd_triage 2024-08-02 01:59:07 UTC
Approved.
Comment 3 Cy Schubert freebsd_committer freebsd_triage 2024-08-02 01:59:48 UTC
Assigning back to dim@ for commit. Approved.
Comment 4 commit-hook freebsd_committer freebsd_triage 2024-08-02 07:54:08 UTC
A commit in branch main references this bug:

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

commit 48aefee896d03d9992ff22a8763c4233849415c0
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-08-01 18:33:37 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-08-02 07:52:10 +0000

    devel/binutils: fix build with libc++ 19

    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 devel/binutils 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,
              |                                          ^
        ./stringpool.h:225:35: note: in instantiation of template class 'std::basic_string<unsigned short>' requested here
          225 |   { return this->add_with_length(s.data(), s.size(), copy, pkey); }
              |                                   ^
        stringpool.cc:530:7: note: in instantiation of member function 'gold::Stringpool_template<unsigned short>::add' requested here
          530 | class Stringpool_template<uint16_t>;
              |       ^
        /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared here
           23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
              |                             ^

    Upstream binutils has fixed this in commit 5e9091dab88 [2], so import it
    as a separate patch, until the next version of binutils is released.

    [1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-removals
    [2] https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=5e9091dab88

    PR:             280563
    Approved by:    cy (maintainer)
    MFH:            2024Q3

 .../binutils/files/patch-commit-5e9091dab88 (new)  | 108 +++++++++++++++++++++
 1 file changed, 108 insertions(+)
Comment 5 commit-hook freebsd_committer freebsd_triage 2024-08-02 08:00:09 UTC
A commit in branch 2024Q3 references this bug:

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

commit 3c10c55a3095550e597a404677c49f358cd6ebee
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-08-01 18:33:37 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-08-02 07:59:04 +0000

    devel/binutils: fix build with libc++ 19

    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 devel/binutils 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,
              |                                          ^
        ./stringpool.h:225:35: note: in instantiation of template class 'std::basic_string<unsigned short>' requested here
          225 |   { return this->add_with_length(s.data(), s.size(), copy, pkey); }
              |                                   ^
        stringpool.cc:530:7: note: in instantiation of member function 'gold::Stringpool_template<unsigned short>::add' requested here
          530 | class Stringpool_template<uint16_t>;
              |       ^
        /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared here
           23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
              |                             ^

    Upstream binutils has fixed this in commit 5e9091dab88 [2], so import it
    as a separate patch, until the next version of binutils is released.

    [1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-removals
    [2] https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=5e9091dab88

    PR:             280563
    Approved by:    cy (maintainer)
    MFH:            2024Q3

    (cherry picked from commit 48aefee896d03d9992ff22a8763c4233849415c0)

 .../binutils/files/patch-commit-5e9091dab88 (new)  | 108 +++++++++++++++++++++
 1 file changed, 108 insertions(+)