Summary: | dev-qt/qtwebengine-5.15.8_p20230313: Fails to build with Clang 16 | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Manuel Nickschas <sputnick> |
Component: | Current packages | Assignee: | Qt Bug Alias <qt> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | chris, darkbasic, gentoo.bugs, gyakovlev, sam |
Priority: | Normal | Keywords: | PATCH |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: |
https://chromium-review.googlesource.com/c/v8/v8/+/3794708 https://bugs.chromium.org/p/chromium/issues/detail?id=1348574 |
||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
build.log
emerge --info qtwebengine-5.15.8_p20230313-clang-16.patch |
Description
Manuel Nickschas
2023-03-30 12:10:08 UTC
Created attachment 859260 [details]
build.log
Created attachment 859261 [details]
emerge --info
Created attachment 859617 [details, diff]
qtwebengine-5.15.8_p20230313-clang-16.patch
Thanks for reporting. A workaround is to save the attached
patch (by me) in the directory:
/etc/portage/patches/dev-qt/qtwebengine-5.15.8_p20230313
(In reply to Mark Wright from comment #3) > Created attachment 859617 [details, diff] [details, diff] > qtwebengine-5.15.8_p20230313-clang-16.patch > > Thanks for reporting. A workaround is to save the attached > patch (by me) in the directory: > /etc/portage/patches/dev-qt/qtwebengine-5.15.8_p20230313 This is probably a fine workaround until Qt 6. Can that patch get merged please? Thanks! Can confirm this is still a problem on 5.15.9_p20230505 (In reply to Sam James from comment #4) > (In reply to Mark Wright from comment #3) > > Created attachment 859617 [details, diff] [details, diff] [details, diff] > > qtwebengine-5.15.8_p20230313-clang-16.patch > > > > Thanks for reporting. A workaround is to save the attached > > patch (by me) in the directory: > > /etc/portage/patches/dev-qt/qtwebengine-5.15.8_p20230313 > > This is probably a fine workaround until Qt 6. This ended up being wrong, by the way. Clang 17 is going to make it so you can't disable the error. *** Bug 910370 has been marked as a duplicate of this bug. *** I just tried this patch for qtwebengine-5.15.10_p20230623. Works for me. I changed the patch name to /etc/portage/patches/dev-qt/qtwebengine\:5 so it's for any slot-5 version. Given qtwebengine:5 won't compile without this, shouldn't it be part of the ebuild now? FWIW, I can't get qtwebengine to compile with gcc either; different problems, but it's still broken. I guess given Sam's comment 7, I'll want to hang on to Clang 16 (until Qt6 hits stable or thereabouts). AFAICT llvm and its friends are slotted, so it should be possible, though quite how to use it - the doesn't seem to be an "eselect llvm" or "eselect clang", but I'm sure it's a Small Matter of Programming. I think backporting https://github.com/v8/v8/commit/d15d49b09dc7aef9edcc4cf6a0cb2b77a0db203f may work. The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6fb7d0a7131b5a5938b12673f4b260d66cc1a527 commit 6fb7d0a7131b5a5938b12673f4b260d66cc1a527 Author: Denis Pronin <dannftk@yandex.ru> AuthorDate: 2023-07-13 14:55:14 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2023-08-04 12:35:02 +0000 dev-qt/qtwebengine: fix compiling with clang-16 clang-16 complains about casting an integer value that exceeds an enumeration capacity in the class base::BitField to not modify chromium code we just cancel warning about this circumstance to prevent it from being converted to an error that fails compilation [sam: Note that this is a bandaid and won't work with Clang 17.] Bug: https://bugs.gentoo.org/903573 Signed-off-by: Denis Pronin <dannftk@yandex.ru> Closes: https://github.com/gentoo/gentoo/pull/31869 Signed-off-by: Sam James <sam@gentoo.org> .../files/qtwebengine-5.15.10_p20230623-clang16.patch | 15 +++++++++++++++ dev-qt/qtwebengine/qtwebengine-5.15.10_p20230623.ebuild | 5 ++++- 2 files changed, 19 insertions(+), 1 deletion(-) |