Closed Bug 576879 Opened 14 years ago Closed 14 years ago

AIX 64bit link error: libxul.so

Categories

(Firefox Build System :: General, defect)

PowerPC
AIX
defect
Not set
normal

Tracking

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

RESOLVED FIXED
Tracking Status
status1.9.2 --- .9-fixed
status1.9.1 --- .12-fixed

People

(Reporter: ul-mcamafia, Assigned: ul-mcamafia)

References

Details

(Keywords: verified1.9.1, verified1.9.2)

Attachments

(3 files)

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.
Status: NEW → ASSIGNED
Assignee: nobody → ul.mcamafia
Instead of adding -bbigtoc flag globally, only add it where needed for avoiding the bigtoc overhead at runtime when possible.
Attachment #455966 - Flags: review?(ted.mielczarek)
Blocks: 576887
Attachment #455966 - Flags: review?(ted.mielczarek) → review?(dtownsend)
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)
Component: XULRunner → Build Config
Product: Toolkit → Core
QA Contact: xulrunner → build-config
Assignee: nobody → ul.mcamafia
Attachment #455966 - Flags: review?(ted.mielczarek) → review+
There were conflicts in the context, recreated today.
Keywords: checkin-needed
Whiteboard: [attachment 461764 to trunk]
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
Risc assessment: Those patches does not alter anything for all officially supported platform. No risc patch.
Attachment #461764 - Flags: approval2.0?
Keywords: checkin-needed
Whiteboard: [attachment 461764 to trunk]
Attachment #455966 - Flags: approval1.9.1.12?
Attachment #461765 - Flags: approval1.9.2.9?
Attachment #455966 - Flags: approval1.9.1.12? → approval1.9.1.12+
Attachment #461765 - Flags: approval1.9.2.9? → approval1.9.2.9+
Approving the fix for the branches.
Keywords: checkin-needed
Whiteboard: [attachment 455966 to branch 1.9.1],[attachment 461765 to branch 1.9.2]
Attachment #461764 - Flags: approval2.0? → approval2.0+
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]
Verified on AIX.
Whiteboard: [attachment 455966 to branch 1.9.1],[attachment 461765 to branch 1.9.2],[attachment 461764 to branch 2.0]
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: