qt5.qtwebengine: Fix build on darwin#358601
Conversation
``` ../../3rdparty/chromium/third_party/harfbuzz-ng/src/src/hb-ft.cc:759:73: error: cast from 'void (*)(FT_Face)' (aka 'void (*)(FT_FaceRec_ *)') to 'FT_Generic_Finalizer' (aka 'void (*)(void *)') converts to incompatible function type [-Werror,-Wcast-function-type-strict] ```
|
I believe that on Linux we devendor |
|
Some weird thing happened.
Grepping the logs of those 2 builds: The green one: The red one: According to: nixpkgs/pkgs/development/libraries/qt-5/5.15/default.nix Lines 284 to 298 in 3917762 Clang 15 should be used, commit ab80059 broke it? @reckenrode |
While grepping the logs, I find I believe |
|
IIRC some of the devendoring flags only function on Linux because of weird Chromium build system stuff. But maybe this one works? |
|
While devendoring is nice, but I think it won't fix the build because:
Need to figure out why |
|
Fair enough. FWIW, I think we won‘t ship Qt 5 WebEngine in 25.05 due to the upcoming EOL, so it might not be worth spending too much time on it anyway. @K900 Maybe we should just work to drop Qt 5 WebEngine early in the cycle? There’s no reasonable expectation that the KDE people are going to be able to keep adequately on top of Chromium security backports, right? |
|
Would be nice, but probably impractical. If someone wants to go and estimate the blast radius, that would be good though. |
|
Well, I feel like the alternative is that we end up having to set it |

Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.