Bug 81750

Summary: [Qt][Mac] ranlib segfaults when creating symbol tables for libWebCore.a.
Product: WebKit Reporter: Zeno Albisser <zeno>
Component: WebKit QtAssignee: Zeno Albisser <zeno>
Status: RESOLVED FIXED    
Severity: Normal CC: vestbo
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch for review.
vestbo: review-
patch for review. - applied changes as requested. none

Description Zeno Albisser 2012-03-21 02:51:28 PDT
ranlib seems not to allow creating symbol tables for archives bigger than 4GB.
We should use some or all of the *AllInOne.cpp files to shrink the library size when building debug on mac.
Comment 1 Zeno Albisser 2012-03-21 05:18:06 PDT
Created attachment 133018 [details]
patch for review.
Comment 2 Tor Arne Vestbø 2012-03-21 09:53:32 PDT
Comment on attachment 133018 [details]
patch for review.

View in context: https://bugs.webkit.org/attachment.cgi?id=133018&action=review

> Source/WebCore/Target.pri:3487
> +        contains(CONFIG, USE_SVG_ALL_IN_ONE) {

If you do use CONFIG then you can use the CONFIG(foo) macro, or even just the foo:bar: syntax.

> Tools/qmake/mkspecs/features/default_pre.prf:127
> +macx {
> +    contains(CONFIG, debug)|contains(CONFIG, debug_and_release) {
> +        message("Building WebKit Debug on mac. Switching to USE_SVG_ALL_IN_ONE configuration.")
> +        CONFIG += USE_SVG_ALL_IN_ONE
> +    }
> +}

You can move this part into features/macx/default_pre.prf, which is specific to mac.  (after loading default_pre).

The message is fine, but prefix it with: root_project_file:message().. otherwise you'll see it for every single project file

You should not have to check for debug_and_release, CONFIG will already contain debug i that case.

For the CONFIG option, please lower-case and generalize it, eg, use_all_in_one_files, that way the config will still apply if we need to add another all-in-one file
Comment 3 Zeno Albisser 2012-03-21 10:37:58 PDT
Created attachment 133071 [details]
patch for review. - applied changes as requested.
Comment 4 Tor Arne Vestbø 2012-03-21 10:53:08 PDT
Comment on attachment 133071 [details]
patch for review. - applied changes as requested.

thanks!
Comment 5 Zeno Albisser 2012-03-21 11:02:28 PDT
Comment on attachment 133071 [details]
patch for review. - applied changes as requested.

Clearing flags on attachment: 133071

Committed r111578: <http://trac.webkit.org/changeset/111578>
Comment 6 Zeno Albisser 2012-03-21 11:02:36 PDT
All reviewed patches have been landed.  Closing bug.