377210 - Linking liblocalmail.dylib on mac-i386 (gcc4) fails on 1.8 branch (phlox tinderbox)

Status: VERIFIED FIXED

(SeaMonkey :: Build Config, defect)

Description

[Robert Kaiser]

Since the fix to bug 366844 has been checked in on 1.8 branch, linking liblocalmail.dylib for i386 (gcc4) fails on the phlox MacOSX tinderbox (actually, the first time the build failed was after it clobbered the first time after this checkin and re-ran configure because of that).

The lines of the actual error are clearly visible in the short log: http://tinderbox.mozilla.org/showlog.cgi?log=Mozilla1.8-SeaMonkey/1176300420.1176312664.22906.gz
Full build log of the same build is here: http://tinderbox.mozilla.org/showlog.cgi?log=Mozilla1.8-SeaMonkey/1176300420.1176312664.22906.gz&fulltext=1

This does only affect SeaMonkey, as it apparently only comes up in shared mail, and Thunderbird uses static mail by default.
The trunk tinderbox, cg-xserve02, manages to link this lib fine.

phlox has OSX and Xcode with all currently available updates.

This is the snippet relevant to the visibility tests from the i386 cross-compile configure run:

-----------
checking for usable wchar_t (2 bytes, unsigned)... no
checking for compiler -fshort-wchar option... yes
checking for visibility(hidden) attribute... yes
checking for visibility(default) attribute... yes
checking for visibility pragma support... no
checking for dirent.h that defines DIR... yes
-----------

Comment 1

[Robert Kaiser]

mailnews/local/build/Makefile.in is where liblocalmail is defined, the two checkins marked green in the following URL are what separates branch from trunk there (1.8 branch is straight rev 1.39, trunk is 1.41): http://bonsai.mozilla.org/cvslog.cgi?file=/mozilla/mailnews/local/build/Makefile.in&rev=HEAD&mark=1.40,1.41

Comment 2

[Benjamin Smedberg]

Attachment: Link rdfutil first, rev. 1

Comment 3

[Robert Kaiser]

Comment on attachment 261414 [details] [diff] [review]
Link rdfutil first, rev. 1

As far as I could test, it looks like it links with that change. Thanks a lot!

Comment 4

[Benjamin Smedberg]

Comment on attachment 261414 [details] [diff] [review]
Link rdfutil first, rev. 1

Requesting approval for linkage bustage fix in mailnews.

Comment 5

[Daniel Veditz [:dveditz]]

Comment on attachment 261414 [details] [diff] [review]
Link rdfutil first, rev. 1

approved for 1.8.1.4, a=dveditz for release-drivers

Comment 6

[Benjamin Smedberg]

Fixed on MOZILLA_1_8_BRANCH

Comment 7

[Robert Kaiser]

phlox links now and spits out nightlies, so verified.