Created attachment 254582 [details] patch Hi, I've recently upgraded to recent head and noticed electron failed to build, due to the clang 19 upgrade. I've created a patch that fixes it for me. Hope it can be useful. All changes are coming from upstream commits: https://github.com/chromium/chromium/commit/23646607e16c63231ae9f49ce5355c270145cf30 https://chromium-review.googlesource.com/c/chromium/src/+/5604664/2/third_party/blink/renderer/platform/wtf/hash_table.h https://android.googlesource.com/platform/external/perfetto.git/+/e2f661907a717551235563389977b7468da6d45e%5E%21/#F0 https://quiche.googlesource.com/quiche/+/c1af894e0f5c4f732a983e7c93227854e203570e https://github.com/electron/electron/commit/8a8241163d4a802a7daeb21e53780f74d2e87cc2#diff-5678864515037a75e2b9c00ec60c3dbbd27f83fc22dd76f07a4b149a70bc04ef
(In reply to Guido Falsi from comment #0) Hi Guido, Thank you for the patch. I have confirmed that the patched port successfully builds with clang 19. I will commit the changes shortly.
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=85e785bd0f4e3e3b979448594b986ee7174acc4a commit 85e785bd0f4e3e3b979448594b986ee7174acc4a Author: Guido Falsi <madpilot@FreeBSD.org> AuthorDate: 2024-10-31 10:53:13 +0000 Commit: Hiroki Tagato <tagattie@FreeBSD.org> CommitDate: 2024-10-31 10:57:00 +0000 devel/electron30: fix build with clang 19 PR: 282370 Reported by: madpilot MFH: 2024Q4 ...c_partition__alloc__base_no__destructor.h (new) | 13 ++++++ .../files/patch-base_containers_id__map.h (new) | 13 ++++++ ...h-electron_shell_common_keyboard__util.cc (new) | 11 +++++ ..._quiche_quic_core_quic__interval__deque.h (new) | 17 +++++++ ...blink_renderer_platform_wtf_hash__table.h (new) | 31 +++++++++++++ ...ing_internal_track__event__data__source.h (new) | 54 ++++++++++++++++++++++ 6 files changed, 139 insertions(+)
A commit in branch 2024Q4 references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=5d2dc5802e1e837ee531c8301b790229159fefa2 commit 5d2dc5802e1e837ee531c8301b790229159fefa2 Author: Guido Falsi <madpilot@FreeBSD.org> AuthorDate: 2024-10-31 10:53:13 +0000 Commit: Hiroki Tagato <tagattie@FreeBSD.org> CommitDate: 2024-10-31 10:58:12 +0000 devel/electron30: fix build with clang 19 PR: 282370 Reported by: madpilot MFH: 2024Q4 (cherry picked from commit 85e785bd0f4e3e3b979448594b986ee7174acc4a) ...c_partition__alloc__base_no__destructor.h (new) | 13 ++++++ .../files/patch-base_containers_id__map.h (new) | 13 ++++++ ...h-electron_shell_common_keyboard__util.cc (new) | 11 +++++ ..._quiche_quic_core_quic__interval__deque.h (new) | 17 +++++++ ...blink_renderer_platform_wtf_hash__table.h (new) | 31 +++++++++++++ ...ing_internal_track__event__data__source.h (new) | 54 ++++++++++++++++++++++ 6 files changed, 139 insertions(+)
Committed, thanks!
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=2457237c15a3db2b496a1e5660e9f0d4a6d56e64 commit 2457237c15a3db2b496a1e5660e9f0d4a6d56e64 Author: Guido Falsi <madpilot@FreeBSD.org> AuthorDate: 2024-11-01 20:50:10 +0000 Commit: Hiroki Tagato <tagattie@FreeBSD.org> CommitDate: 2024-11-01 20:51:19 +0000 devel/electron31: fix build with clang 19 PR: 282370 Reported by: madpilot MFH: 2024Q4 ...h-electron_shell_common_keyboard__util.cc (new) | 11 +++++ ...blink_renderer_platform_wtf_hash__table.h (new) | 31 +++++++++++++ ...ing_internal_track__event__data__source.h (new) | 54 ++++++++++++++++++++++ 3 files changed, 96 insertions(+)
A commit in branch 2024Q4 references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=cf43f81e2d25d27dc7f3e0f7fc54ed9f4c68350f commit cf43f81e2d25d27dc7f3e0f7fc54ed9f4c68350f Author: Guido Falsi <madpilot@FreeBSD.org> AuthorDate: 2024-11-01 20:50:10 +0000 Commit: Hiroki Tagato <tagattie@FreeBSD.org> CommitDate: 2024-11-01 20:52:17 +0000 devel/electron31: fix build with clang 19 PR: 282370 Reported by: madpilot MFH: 2024Q4 (cherry picked from commit 2457237c15a3db2b496a1e5660e9f0d4a6d56e64) ...h-electron_shell_common_keyboard__util.cc (new) | 11 +++++ ...blink_renderer_platform_wtf_hash__table.h (new) | 31 +++++++++++++ ...ing_internal_track__event__data__source.h (new) | 54 ++++++++++++++++++++++ 3 files changed, 96 insertions(+)