Bug 81750 - [Qt][Mac] ranlib segfaults when creating symbol tables for libWebCore.a.
Summary: [Qt][Mac] ranlib segfaults when creating symbol tables for libWebCore.a.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Zeno Albisser
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-21 02:51 PDT by Zeno Albisser
Modified: 2012-03-21 11:02 PDT (History)
1 user (show)

See Also:


Attachments
patch for review. (16.66 KB, patch)
2012-03-21 05:18 PDT, Zeno Albisser
vestbo: review-
Details | Formatted Diff | Diff
patch for review. - applied changes as requested. (16.58 KB, patch)
2012-03-21 10:37 PDT, Zeno Albisser
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.