AIX 64bit link error: libxul.so

RESOLVED FIXED

Status

()

Core
Build Config
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: ul, Assigned: ul)

Tracking

({verified1.9.1, verified1.9.2})

unspecified
PowerPC
AIX
verified1.9.1, verified1.9.2
Points:
---

Firefox Tracking Flags

(status1.9.2 .9-fixed, status1.9.1 .12-fixed)

Details

Attachments

(3 attachments)

(Assignee)

Description

7 years ago
When linking obj-dir/toolkit/library/libxul.so
the subsequent error appears:

../staticlib/libmorkreader_s.a ../../staticlib/libgtkxtbin.a ../../staticlib/libgfxpsshar.a ../../staticlib/libthebes.a ../../staticlib/libjsj.a   -L../../dist/lib -lsqlite3 -L../../dist/bin -L../../dist/lib  -L../../dist/bin -L../../dist/lib -L../../jpeg -lmozjpeg -L../../modules/libimg/png -lmozpng ../../gfx/qcms/libmozqcms.a -L/home/ulink/src/comm-1.9.1/mozilla/obj-fx35-64/dist/bin -lmozjs -L../../dist/bin -L../../dist/lib -lcrmf -lsmime3 -lssl3 -lnss3 -lnssutil3 -L/opt/freeware/lib -lcairo -lfreetype -lfontconfig    -L/opt/freeware/lib -lXrender -lSM -lICE -lX11 -lcairo    -lz  -L/home/ulink/src/comm-1.9.1/mozilla/obj-fx35-64/dist/lib -lplds4 -lplc4 -lnspr4 -lpthreads -ldl  -lX11  -L/opt/freeware/lib -lpangoft2-1.0 -lpango-1.0 -lgobject-2.0 -lgmodule-2.0 -lglib-2.0 -lintl -liconv   -L/opt/freeware/lib -lgtk-x11-2.0 -latk-1.0 -lgdk-x11-2.0 -lgdk_pixbuf-2.0 -lm -lpangocairo-1.0 -lpango-1.0 -lcairo -lgobject-2.0 -lgmodule-2.0 -lglib-2.0 -lintl -liconv   -lXm -lXt -lgthread-2.0 -L/opt/freeware/lib -lfreetype -lz  -lC_r -ldl -lm -lc_r  -liconv   
ld: 0711-781 ERROR: TOC overflow. TOC size: 131416      Maximum size: 65536
gmake[3]: *** [libxul.so] Error 12

The 32bit compilation on AIX is not affected.
Neither SeaMonkey nor Thunderbird are affected today. Firefox only.

The linker needs the additional -bbigtoc flag passed.
(Assignee)

Updated

7 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

7 years ago
Assignee: nobody → ul.mcamafia
(Assignee)

Comment 1

7 years ago
Created attachment 455966 [details] [diff] [review]
Adding needed ld flag for AIX 64bit build branch 1.9.1 patch

Instead of adding -bbigtoc flag globally, only add it where needed for avoiding the bigtoc overhead at runtime when possible.
(Assignee)

Updated

7 years ago
Attachment #455966 - Flags: review?(ted.mielczarek)
(Assignee)

Updated

7 years ago
Blocks: 576887
(Assignee)

Updated

7 years ago
Attachment #455966 - Flags: review?(ted.mielczarek) → review?(dtownsend)
(Assignee)

Updated

7 years ago
Assignee: ul.mcamafia → nobody
Component: Build Config → XULRunner
Product: Core → Toolkit
QA Contact: build-config → xulrunner
Comment on attachment 455966 [details] [diff] [review]
Adding needed ld flag for AIX 64bit build branch 1.9.1 patch

I can't review this.
Attachment #455966 - Flags: review?(dtownsend) → review?(ted.mielczarek)

Updated

7 years ago
Component: XULRunner → Build Config
Product: Toolkit → Core
QA Contact: xulrunner → build-config
Assignee: nobody → ul.mcamafia
Attachment #455966 - Flags: review?(ted.mielczarek) → review+
(Assignee)

Comment 3

7 years ago
Created attachment 461764 [details] [diff] [review]
Same patch recreated for trunk

There were conflicts in the context, recreated today.
(Assignee)

Comment 4

7 years ago
Created attachment 461765 [details] [diff] [review]
Only with context adjusted for branch 1.9.2
(Assignee)

Updated

7 years ago
Keywords: checkin-needed
Whiteboard: [attachment 461764 to trunk]
(Assignee)

Updated

7 years ago
Attachment #455966 - Attachment description: Adding needed ld flag for AIX 64bit build → Adding needed ld flag for AIX 64bit build branch 1.9.1 patch
(Assignee)

Comment 5

7 years ago
Risc assessment: Those patches does not alter anything for all officially supported platform. No risc patch.
(Assignee)

Updated

7 years ago
Attachment #461764 - Flags: approval2.0?
(Assignee)

Updated

7 years ago
Keywords: checkin-needed
Whiteboard: [attachment 461764 to trunk]
(Assignee)

Updated

7 years ago
Attachment #455966 - Flags: approval1.9.1.12?
(Assignee)

Updated

7 years ago
Attachment #461765 - Flags: approval1.9.2.9?

Updated

7 years ago
Attachment #455966 - Flags: approval1.9.1.12? → approval1.9.1.12+

Updated

7 years ago
Attachment #461765 - Flags: approval1.9.2.9? → approval1.9.2.9+

Comment 6

7 years ago
Approving the fix for the branches.
(Assignee)

Updated

7 years ago
Keywords: checkin-needed
Whiteboard: [attachment 455966 to branch 1.9.1],[attachment 461765 to branch 1.9.2]

Updated

7 years ago
Attachment #461764 - Flags: approval2.0? → approval2.0+
(Assignee)

Updated

7 years ago
Whiteboard: [attachment 455966 to branch 1.9.1],[attachment 461765 to branch 1.9.2] → [attachment 455966 to branch 1.9.1],[attachment 461765 to branch 1.9.2],[attachment 461764 to branch 2.0]
http://hg.mozilla.org/mozilla-central/rev/1d913becdd1a
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/0ee7204effe9
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/1a98f56f8777
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
status1.9.1: --- → .12-fixed
status1.9.2: --- → .9-fixed
Resolution: --- → FIXED
Keywords: checkin-needed
(Assignee)

Comment 8

7 years ago
Verified on AIX.
Keywords: verified1.9.1, verified1.9.2
Whiteboard: [attachment 455966 to branch 1.9.1],[attachment 461765 to branch 1.9.2],[attachment 461764 to branch 2.0]
You need to log in before you can comment on or make changes to this bug.