Last Comment Bug 673616 - --enable-shared-js + --enable-shark breaks xpcom test build
: --enable-shared-js + --enable-shark breaks xpcom test build
Product: Core
Classification: Components
Component: XPCOM (show other bugs)
: unspecified
: x86_64 Linux
-- normal (vote)
: mozilla8
Assigned To: Nobody; OK to take it and work on it
: Nathan Froyd [:froydnj]
Depends on:
Blocks: 673631
  Show dependency treegraph
Reported: 2011-07-22 16:55 PDT by Steve Fink [:sfink] [:s:]
Modified: 2011-11-30 02:04 PST (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Add JS libs into link of tests (863 bytes, patch)
2011-07-22 16:55 PDT, Steve Fink [:sfink] [:s:]
khuey: review+
Details | Diff | Splinter Review

Description User image Steve Fink [:sfink] [:s:] 2011-07-22 16:55:19 PDT
Created attachment 547844 [details] [diff] [review]
Add JS libs into link of tests

xpcom/tests/TestHarness.h conditionally #includes "jsdbgapi.h" if MOZ_SHARK is defined, so that it can call JS_StartProfiling. (eg) is not included on the link of the tests that use TestHarness.h.

khuey says the attached is the right fix. I don't really understand why it was working without --enable-shared-js, and if it was because the JS lib was included in, then I don't understand why this wouldn't cause duplicate symbols or something. I'm hoping I'll never have to understand.

I am changing the profiling stuff around substantially. This fix will be necessary to avoid --enable-shared-js *always* breaking the test builds, which would suck.
Comment 1 User image Kyle Huey [:khuey] (Exited; not receiving bugmail, email if necessary) 2011-07-25 10:30:24 PDT
Comment on attachment 547844 [details] [diff] [review]
Add JS libs into link of tests

NB: You need to make the same change to js/src/config/, or you will anger the copy-paste test in make check.
Comment 2 User image Steve Fink [:sfink] [:s:] 2011-08-01 16:07:59 PDT
Mirrored the changes to js/src, thanks.
Comment 3 User image Marco Bonardo [::mak] 2011-08-02 03:22:27 PDT
Comment 4 User image Mike Hommey [:glandium] 2011-11-30 02:04:20 PST
For what it's worth, the fix for this bug is wrong and broke win64 on the landing of bug 702179. The issue will be resolved there.

Note You need to log in before you can comment on or make changes to this bug.