Bug 282165 - devel/librevisa: fix build with libc++ 19
Summary: devel/librevisa: 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: freebsd-ports-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks: 280562
  Show dependency treegraph
 
Reported: 2024-10-17 19:18 UTC by Dimitry Andric
Modified: 2024-10-30 18:31 UTC (History)
1 user (show)

See Also:


Attachments
devel/librevisa: fix build with libc++ 19 (2.58 KB, patch)
2024-10-17 19:22 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-10-17 19:18:37 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/librevisa 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,
        |                                          ^
  usb/usb_resource_creator.cpp:73:20: note: in instantiation of template class 'std::basic_string<unsigned short>' requested here
     73 |         usb_string serial;
        |                    ^
  /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared here
     23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
        |                             ^

These can be fixed by using `char16_t` instead of `uint16_t` for the
`usb_string` type.

[1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-removals
Comment 1 Dimitry Andric freebsd_committer freebsd_triage 2024-10-17 19:22:53 UTC
Created attachment 254315 [details]
devel/librevisa: fix build with libc++ 19
Comment 2 Robert Clausecker freebsd_committer freebsd_triage 2024-10-26 11:29:42 UTC
Patch looks fine.
Port is unmaintained (but this would fall under the build fix blanket anyway).
Comment 3 commit-hook freebsd_committer freebsd_triage 2024-10-30 18:28:53 UTC
A commit in branch main references this bug:

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

commit df7422d81ae570a17b48662af9f20f464f12b076
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-10-17 14:28:46 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-10-30 18:22:30 +0000

    devel/librevisa: 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/librevisa 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,
            |                                          ^
      usb/usb_resource_creator.cpp:73:20: note: in instantiation of template class 'std::basic_string<unsigned short>' requested here
         73 |         usb_string serial;
            |                    ^
      /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared here
         23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
            |                             ^

    These can be fixed by using `char16_t` instead of `uint16_t` for the
    `usb_string` type.

    [1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-removals

    PR:             282165
    MFH:            2024Q4

 .../files/patch-src_usb_usb__resource__creator.cpp (new)      | 11 +++++++++++
 devel/librevisa/files/patch-src_usb_usb__string.h (new)       | 11 +++++++++++
 2 files changed, 22 insertions(+)
Comment 4 commit-hook freebsd_committer freebsd_triage 2024-10-30 18:29:54 UTC
A commit in branch 2024Q4 references this bug:

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

commit 00dce14954e15cd7bf837456ab396b5d21da29b8
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-10-17 14:28:46 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-10-30 18:29:00 +0000

    devel/librevisa: 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/librevisa 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,
            |                                          ^
      usb/usb_resource_creator.cpp:73:20: note: in instantiation of template class 'std::basic_string<unsigned short>' requested here
         73 |         usb_string serial;
            |                    ^
      /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared here
         23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
            |                             ^

    These can be fixed by using `char16_t` instead of `uint16_t` for the
    `usb_string` type.

    [1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-removals

    PR:             282165
    MFH:            2024Q4
    (cherry picked from commit df7422d81ae570a17b48662af9f20f464f12b076)

 .../files/patch-src_usb_usb__resource__creator.cpp (new)      | 11 +++++++++++
 devel/librevisa/files/patch-src_usb_usb__string.h (new)       | 11 +++++++++++
 2 files changed, 22 insertions(+)