Bug 282645 - graphics/povray37: fix build with libc++ 19
Summary: graphics/povray37: 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-11-09 12:04 UTC by Dimitry Andric
Modified: 2024-11-21 23:13 UTC (History)
1 user (show)

See Also:
bsd: maintainer-feedback+


Attachments
graphics/povray37: fix build with libc++ 19 (5.01 KB, patch)
2024-11-09 12:05 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-09 12:04:06 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 graphics/povray37 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,
        |                                          ^
  ./base/configbase.h:159:19: note: in instantiation of template class 'std::basic_string<unsigned short>' requested here
    159 | inline UCS2String ASCIItoUCS2String(const char *s)
        |                   ^
  /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 `POVMSUCS2` as `char16_t` instead of
`unsigned short`, `UCS2` as `char16_t` instead of `unsigned short`, and
`UCS4` as `char32_t` instead of `unsigned int` in the passed-in
`CPPFLAGS` variable.

Also, patch up `vfe.cpp` and `vfeconf.h` to use `char16_t` and
`u16string` instead of the non-standard types.

[1] https://libcxx.llvm.org/ReleaseNotes/19.html#deprecations-and-removals
Comment 1 Dimitry Andric freebsd_committer freebsd_triage 2024-11-09 12:05:44 UTC
Created attachment 255048 [details]
graphics/povray37: fix build with libc++ 19
Comment 2 Philipp Ost 2024-11-21 22:27:54 UTC
Thanks for taking care of this. I trust everything runs fine, so please go ahead with committing it.
Comment 3 commit-hook freebsd_committer freebsd_triage 2024-11-21 23:10:34 UTC
A commit in branch main references this bug:

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

commit 6133473e4227abbfcf023bea6ab5eeed9c17e55b
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-11-09 12:04:23 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-11-21 23:06:15 +0000

    graphics/povray37: 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 graphics/povray37 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,
            |                                          ^
      ./base/configbase.h:159:19: note: in instantiation of template class 'std::basic_string<unsigned short>' requested here
        159 | inline UCS2String ASCIItoUCS2String(const char *s)
            |                   ^
      /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 `POVMSUCS2` as `char16_t` instead of
    `unsigned short`, `UCS2` as `char16_t` instead of `unsigned short`, and
    `UCS4` as `char32_t` instead of `unsigned int` in the passed-in
    `CPPFLAGS` variable.

    Also, patch up `vfe.cpp` and `vfeconf.h` to use `char16_t` and
    `u16string` instead of the non-standard types.

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

    PR:             282645
    Approved by:    bsd@philippost.de (maintainer)
    MFH:            2024Q4

 graphics/povray37/Makefile                        |  3 +++
 graphics/povray37/files/patch-vfe_vfe.cpp (new)   | 26 +++++++++++++++++++++++
 graphics/povray37/files/patch-vfe_vfeconf.h (new) | 19 +++++++++++++++++
 3 files changed, 48 insertions(+)
Comment 4 commit-hook freebsd_committer freebsd_triage 2024-11-21 23:12:35 UTC
A commit in branch 2024Q4 references this bug:

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

commit fa64477e6c3f80ff90f2e661fdd1a2c0de0ee8b2
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2024-11-09 12:04:23 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2024-11-21 23:11:13 +0000

    graphics/povray37: 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 graphics/povray37 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,
            |                                          ^
      ./base/configbase.h:159:19: note: in instantiation of template class 'std::basic_string<unsigned short>' requested here
        159 | inline UCS2String ASCIItoUCS2String(const char *s)
            |                   ^
      /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 `POVMSUCS2` as `char16_t` instead of
    `unsigned short`, `UCS2` as `char16_t` instead of `unsigned short`, and
    `UCS4` as `char32_t` instead of `unsigned int` in the passed-in
    `CPPFLAGS` variable.

    Also, patch up `vfe.cpp` and `vfeconf.h` to use `char16_t` and
    `u16string` instead of the non-standard types.

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

    PR:             282645
    Approved by:    bsd@philippost.de (maintainer)
    MFH:            2024Q4

    (cherry picked from commit 6133473e4227abbfcf023bea6ab5eeed9c17e55b)

 graphics/povray37/Makefile                        |  3 +++
 graphics/povray37/files/patch-vfe_vfe.cpp (new)   | 26 +++++++++++++++++++++++
 graphics/povray37/files/patch-vfe_vfeconf.h (new) | 19 +++++++++++++++++
 3 files changed, 48 insertions(+)