Bug 282524 - science/py-dlib: fix build with libc++ 19
Summary: science/py-dlib: 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: Gleb Popov
URL:
Keywords:
Depends on:
Blocks: 280562
  Show dependency treegraph
 
Reported: 2024-11-03 20:33 UTC by Dimitry Andric
Modified: 2024-11-04 07:44 UTC (History)
0 users

See Also:
arrowd: maintainer-feedback+


Attachments
science/py-dlib: fix build with libc++ 19 (2.75 KB, patch)
2024-11-03 20:34 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-11-03 20:33:23 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 science/py-dlib 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 int>'
    820 |   static_assert(is_same<_CharT, typename traits_type::char_type>::value,
        |                                          ^
  /wrkdirs/usr/ports/science/py-dlib/work-py311/dlib-19.24.6/dlib/bigint/../unicode/unicode.h:718:19: note: in instantiation of template class 'std::basic_string<unsigned int>' requested here
    718 |     const ustring convert_to_utf32 (
        |                   ^
  /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared here
     23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
        |                             ^

This can be fixed by defining `unichar` as `char32_t` instead of
`uint32` in `unicode.h`, and adding a serializer for `char32_t` to
`serialize.h`.

[1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-removals
Comment 1 Dimitry Andric freebsd_committer freebsd_triage 2024-11-03 20:34:58 UTC
Created attachment 254916 [details]
science/py-dlib: fix build with libc++ 19
Comment 2 Gleb Popov freebsd_committer freebsd_triage 2024-11-04 04:44:22 UTC
Thank you for dealing with this. I tried to do it myself but didn't figure the USE_DEFAULT_INT_SERIALIZATION_FOR() part. I can also push this change for you, if you want.
Comment 3 commit-hook freebsd_committer freebsd_triage 2024-11-04 07:38:36 UTC
A commit in branch main references this bug:

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

commit e494e799b8f4976ab09b60d606f5e9ee6e3747c2
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-11-03 20:33:53 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-11-04 07:37:27 +0000

    science/py-dlib: 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 science/py-dlib 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 int>'
        820 |   static_assert(is_same<_CharT, typename traits_type::char_type>::value,
            |                                          ^
      /wrkdirs/usr/ports/science/py-dlib/work-py311/dlib-19.24.6/dlib/bigint/../unicode/unicode.h:718:19: note: in instantiation of template class 'std::basic_string<unsigned int>' requested here
        718 |     const ustring convert_to_utf32 (
            |                   ^
      /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared here
         23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
            |                             ^

    This can be fixed by defining `unichar` as `char32_t` instead of
    `uint32` in `unicode.h`, and adding a serializer for `char32_t` to
    `serialize.h`.

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

    PR:             282524
    Approved by:    arrowd (maintainer)
    MFH:            2024Q4

 science/py-dlib/files/patch-dlib_serialize.h (new)       | 11 +++++++++++
 science/py-dlib/files/patch-dlib_unicode_unicode.h (new) | 11 +++++++++++
 2 files changed, 22 insertions(+)
Comment 4 commit-hook freebsd_committer freebsd_triage 2024-11-04 07:39:37 UTC
A commit in branch 2024Q4 references this bug:

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

commit 1ea91c341e2e88a911f9fb07f608c11643c77938
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-11-03 20:33:53 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-11-04 07:38:35 +0000

    science/py-dlib: 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 science/py-dlib 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 int>'
        820 |   static_assert(is_same<_CharT, typename traits_type::char_type>::value,
            |                                          ^
      /wrkdirs/usr/ports/science/py-dlib/work-py311/dlib-19.24.6/dlib/bigint/../unicode/unicode.h:718:19: note: in instantiation of template class 'std::basic_string<unsigned int>' requested here
        718 |     const ustring convert_to_utf32 (
            |                   ^
      /usr/include/c++/v1/__fwd/string.h:23:29: note: template is declared here
         23 | struct _LIBCPP_TEMPLATE_VIS char_traits;
            |                             ^

    This can be fixed by defining `unichar` as `char32_t` instead of
    `uint32` in `unicode.h`, and adding a serializer for `char32_t` to
    `serialize.h`.

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

    PR:             282524
    Approved by:    arrowd (maintainer)
    MFH:            2024Q4

    (cherry picked from commit e494e799b8f4976ab09b60d606f5e9ee6e3747c2)

 science/py-dlib/files/patch-dlib_serialize.h (new)       | 11 +++++++++++
 science/py-dlib/files/patch-dlib_unicode_unicode.h (new) | 11 +++++++++++
 2 files changed, 22 insertions(+)