487694 - unable to build ssltunnel and some xpcom/test files due to CxxFrameHandler linkage

Status: VERIFIED FIXED

(Core :: XPCOM, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

This is due to bug 481579 where we use /NODEFAULTLIB by default.  The problem is in files like ssltunnel.cpp we use STL for <vector> and <string> which internally require SEH.  During link time, we don't have an exception handler available so we bail out.

The solution which I am using is to extract stdexcpt.obj from libcpmt.lib and create a separate .lib:
C:\PROGRA~1\MICROS~1.0\VC\ce\lib\armv4i>..\..\bin\x86_arm\lib.exe /extract:objr\armv4i\stdexcpt.obj /out:stdexcpt.lib libcpmt.lib

Then we need to modify the makefile:
http://mxr.mozilla.org/mozilla-central/source/testing/mochitest/ssltunnel/Makefile.in

We also need to do it here as well (nsIFileEnumerator.cpp won't link):
http://mxr.mozilla.org/mozilla-central/source/xpcom/tests/Makefile.in

To include the stdexcpt.lib file that we created.

I will update this bug with other edits that we require to get a build working.  

The final patch will be a set of makefile.in tweaks as well as a .lib file.  Or we can generate the .lib file dynamically which is safer if we are going to be building on different sdk versions as time goes on.

Comment 1

[Hiroyuki Ikezoe (:hiro)]

Attachment: Patch for ssltunnel.exe

Set libcmt.lib into EXTRA_LIBS expliciltly.

Comment 2

[Hiroyuki Ikezoe (:hiro)]

Attachment: Patch for nsIFileEnumerator.exe

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

I verified these patches will allow --enable-tests to build on wince

Comment 4

[Jeff Walden [:Waldo]]

Comment on attachment 376175 [details] [diff] [review]
Patch for ssltunnel.exe

rs=me if it works, I have zero knowledge about how non-trivial build processes work and would punt to anyone else if this weren't so small...

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

blassey, do you have any input on these patches?  Otherwise I would like to get these checked in.

Comment 6

[Brad Lassey [:blassey] (use needinfo?)]

those look fine to me.

Comment 7

[Reed Loden [:reed]]

http://hg.mozilla.org/mozilla-central/rev/8bdc45653040
http://hg.mozilla.org/mozilla-central/rev/32f180340f3a

Comment 8

[Joel Maher ( :jmaher ) (UTC -8)]

verified with 1.9.2 build and winmo alpha3