Bug 281975 - science/dlib-cpp: fix build with clang and libc++ 19
Summary: science/dlib-cpp: fix build with clang and 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-10-09 18:59 UTC by Dimitry Andric
Modified: 2024-10-09 19:18 UTC (History)
0 users

See Also:
arrowd: maintainer-feedback+


Attachments
science/dlib-cpp: fix build with clang and libc++ 19 (4.48 KB, patch)
2024-10-09 19:04 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-09 18:59:45 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/dlib-cpp 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/dlib-cpp/work/dlib-19.24/dlib/bigint/../unicode/unicode.h:496:26: note: in instantiation of template class 'std::basic_string<unsigned int>' requested here
    496 |     inline const ustring convert_utf8_to_utf32 (
        |                          ^
  /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 `char32_t` instead of `uint32_t` for the
`unichar` type.

Also, clang 19 now implements CWG 96 [1], which requires a template
argument list after a 'template' keyword, resulting in errors similar
to:

  /wrkdirs/usr/ports/science/dlib-cpp/work/dlib-19.24/dlib/filtering/../global_optimization/find_max_global.h:61:130: error: a template argument list is expected after a name prefixed by the template keyword [-Wmissing-template-arg-list-after-template-kw]
     61 |             static auto go(T&& f, const matrix<double,0,1>& a) -> decltype(call_function_and_expand_args<max_unpack-1>::template go(std::forward<T>(f),a))
        |                                                                                                                                  ^

This can be fixed by adding `<>` after the `go` identifier.

[1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-removals
Comment 1 Dimitry Andric freebsd_committer freebsd_triage 2024-10-09 19:04:34 UTC
Created attachment 254121 [details]
science/dlib-cpp: fix build with clang and libc++ 19
Comment 2 Gleb Popov freebsd_committer freebsd_triage 2024-10-09 19:07:52 UTC
LGTM.
Comment 3 commit-hook freebsd_committer freebsd_triage 2024-10-09 19:11:25 UTC
A commit in branch main references this bug:

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

commit e8b1dca33760dc680dcc732dcd3e9c7be1d79e76
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-10-08 10:28:32 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-10-09 19:10:15 +0000

    science/dlib-cpp: fix build with clang and 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/dlib-cpp 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/dlib-cpp/work/dlib-19.24/dlib/bigint/../unicode/unicode.h:496:26: note: in instantiation of template class 'std::basic_string<unsigned int>' requested here
        496 |     inline const ustring convert_utf8_to_utf32 (
            |                          ^
      /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 `char32_t` instead of `uint32_t` for the
    `unichar` type.

    Also, clang 19 now implements CWG 96 [1], which requires a template
    argument list after a 'template' keyword, resulting in errors similar
    to:

      /wrkdirs/usr/ports/science/dlib-cpp/work/dlib-19.24/dlib/filtering/../global_optimization/find_max_global.h:61:130: error: a template argument list is expected after a name prefixed by the template keyword [-Wmissing-template-arg-list-after-template-kw]
         61 |             static auto go(T&& f, const matrix<double,0,1>& a) -> decltype(call_function_and_expand_args<max_unpack-1>::template go(std::forward<T>(f),a))
            |                                                                                                                                  ^

    This can be fixed by adding `<>` after the `go` identifier.

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

    PR:             281975
    Approved by:    arrowd (maintainer)
    MFH:            2024Q3

 .../patch-dlib_global__optimization_find__max__global.h (new) | 11 +++++++++++
 science/dlib-cpp/files/patch-dlib_serialize.h (new)           | 10 ++++++++++
 science/dlib-cpp/files/patch-dlib_unicode_unicode.h (new)     | 11 +++++++++++
 3 files changed, 32 insertions(+)
Comment 4 commit-hook freebsd_committer freebsd_triage 2024-10-09 19:17:26 UTC
A commit in branch 2024Q4 references this bug:

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

commit b67619a7aa9c10ff28ba0ff7067c4737509c872f
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-10-08 10:28:32 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-10-09 19:16:48 +0000

    science/dlib-cpp: fix build with clang and 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/dlib-cpp 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/dlib-cpp/work/dlib-19.24/dlib/bigint/../unicode/unicode.h:496:26: note: in instantiation of template class 'std::basic_string<unsigned int>' requested here
        496 |     inline const ustring convert_utf8_to_utf32 (
            |                          ^
      /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 `char32_t` instead of `uint32_t` for the
    `unichar` type.

    Also, clang 19 now implements CWG 96 [1], which requires a template
    argument list after a 'template' keyword, resulting in errors similar
    to:

      /wrkdirs/usr/ports/science/dlib-cpp/work/dlib-19.24/dlib/filtering/../global_optimization/find_max_global.h:61:130: error: a template argument list is expected after a name prefixed by the template keyword [-Wmissing-template-arg-list-after-template-kw]
         61 |             static auto go(T&& f, const matrix<double,0,1>& a) -> decltype(call_function_and_expand_args<max_unpack-1>::template go(std::forward<T>(f),a))
            |                                                                                                                                  ^

    This can be fixed by adding `<>` after the `go` identifier.

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

    PR:             281975
    Approved by:    arrowd (maintainer)
    MFH:            2024Q3

    (cherry picked from commit e8b1dca33760dc680dcc732dcd3e9c7be1d79e76)

 .../patch-dlib_global__optimization_find__max__global.h (new) | 11 +++++++++++
 science/dlib-cpp/files/patch-dlib_serialize.h (new)           | 10 ++++++++++
 science/dlib-cpp/files/patch-dlib_unicode_unicode.h (new)     | 11 +++++++++++
 3 files changed, 32 insertions(+)