Summary: | multimedia/phonon: fix build with clang 16 | ||||||
---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Dimitry Andric <dim> | ||||
Component: | Individual Port(s) | Assignee: | freebsd-kde (group) <kde> | ||||
Status: | Closed FIXED | ||||||
Severity: | Affects Some People | CC: | arrowd, rakuco | ||||
Priority: | --- | Flags: | bugzilla:
maintainer-feedback?
(kde) |
||||
Version: | Latest | ||||||
Hardware: | Any | ||||||
OS: | Any | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 271047 | ||||||
Attachments: |
|
Description
Dimitry Andric
![]() ![]() Created attachment 242814 [details]
multimedia/phonon: fix build with clang 16
Note that clang upstream plans to make -Wenum-constexpr-conversion a hard error in the future (i.e it cannot be turned off anymore), so at some point actually fixing the problem should be done. In that case, the above patch still applies. :) For the record, upstream ended up doing the same thing that's been done in https://cgit.freebsd.org/ports/commit/?id=83e1d12fd0838b9cb38867f54bef20790e05b3af I've posted a comment to https://invent.kde.org/libraries/phonon/-/merge_requests/13 pointing out what dim's mentioned here, hopefully this leads to a proper fix there. (In reply to Raphael Kubo da Costa from comment #4) Point them to https://reviews.llvm.org/D150226, where it seems like the possibility of disabling this error is going to go away after clang 17. I have submitted https://invent.kde.org/libraries/phonon/-/merge_requests/16 for the future-proof fix. A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=73b6e876a9d3bb44fb7560c5efd99f9fe5f334b7 commit 73b6e876a9d3bb44fb7560c5efd99f9fe5f334b7 Author: Raphael Kubo da Costa <rakuco@FreeBSD.org> AuthorDate: 2023-08-05 15:01:50 +0000 Commit: Raphael Kubo da Costa <rakuco@FreeBSD.org> CommitDate: 2023-08-05 15:03:07 +0000 multimedia/phonon: Switch to upstream's fix for clang 16 build issues Instead of passing a compiler flag to work around a clang 16 build issue, adopt the fix that dim@ landed upstream that fixes the code in question. PR: 272029 Approved by: kde (arrowd) Differential Revision: https://reviews.freebsd.org/D41307 multimedia/phonon/Makefile | 9 ++---- ...a3eb81bcbc730e44c27a0d060a85cf9213d.patch (new) | 36 ++++++++++++++++++++++ 2 files changed, 38 insertions(+), 7 deletions(-) |