Bug 274032 - [GTK] [2.45.1] Fails to build on big-endian machines due to lack of support in Skia
Summary: [GTK] [2.45.1] Fails to build on big-endian machines due to lack of support i...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Linux
: P2 Normal
Assignee: Diego Pino
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-05-11 01:51 PDT by Alberto Garcia
Modified: 2024-05-14 09:16 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alberto Garcia 2024-05-11 01:51:10 PDT
https://github.com/WebKit/WebKit/blob/webkitgtk-2.45.1/Source/ThirdParty/skia/include/private/base/SkLoadUserConfig.h#L56

#if defined(SK_CPU_BENDIAN) && !defined(I_ACKNOWLEDGE_SKIA_DOES_NOT_SUPPORT_BIG_ENDIAN)
    #error "The Skia team is not endian-savvy enough to support big-endian CPUs."
    #error "If you still want to use Skia,"
    #error "please define I_ACKNOWLEDGE_SKIA_DOES_NOT_SUPPORT_BIG_ENDIAN."
#endif

I confirm that it builds on s390x at least if I use -DUSE_SKIA=OFF so we should autodetect that situation.
Comment 1 Michael Catanzaro 2024-05-11 05:11:00 PDT
Pull request: https://github.com/WebKit/WebKit/pull/28429
Comment 2 EWS 2024-05-14 00:36:18 PDT
Committed 278740@main (7a1468c7a3f3): <https://commits.webkit.org/278740@main>

Reviewed commits have been landed. Closing PR #28429 and removing active labels.
Comment 3 Philippe Normand 2024-05-14 05:13:06 PDT
This broke builds where CMake is too old (3.16)...

CMake Error at Source/cmake/OptionsWPE.cmake:84 (if):                                                                                                                                                              
  if given arguments:                                                                                                                                                                                              
                                                                                                                                                                                                                   
    "3.16.9" "VERSION_LESS" "3.20" "OR" "STREQUAL" "LITTLE_ENDIAN"                                                                                                                                                 
                                                                                                                                                                                                                   
  Unknown arguments specified
Comment 4 Michael Catanzaro 2024-05-14 06:04:02 PDT
Drat, I forgot we have to do it in two separate conditionals. I'll land a follow-up.
Comment 5 Diego Pino 2024-05-14 09:11:39 PDT
Re-opening for pull request https://github.com/webkit/webkit/pull/28540
Comment 6 Diego Pino 2024-05-14 09:14:00 PDT
Many post-commit bots are currently failing and there's another build error that needs to be fixed.

https://build.webkit.org/#/builders/992
Comment 7 EWS 2024-05-14 09:16:54 PDT
Committed 278764@main (9c1f32d066a4): <https://commits.webkit.org/278764@main>

Reviewed commits have been landed. Closing PR #28540 and removing active labels.