Closed Bug 1135493 Opened 10 years ago Closed 2 years ago

javascript engine unified library

Categories

(Core :: JavaScript Engine, defect)

36 Branch
x86_64
Windows 8.1
defect
Not set
normal

Tracking

()

RESOLVED INCOMPLETE

People

(Reporter: steve, Unassigned)

Details

User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:36.0) Gecko/20100101 Firefox/36.0 Build ID: 20150217132925 Steps to reproduce: Tried linking our external app to mozilla javascript engine - ff36b10 source Actual results: couldn't find an obvious library to link to ... inspected js.exe compile/bind from full mozilla build to identify required dependencies ... pulled out what required & bound app to those - below for reference. Psapi.lib // For GetProcessMemoryInfo - currently required // Mozilla JS dependencies ... js_static.lib [moz_build]\mozglue\crt\mozcrt.lib [moz_build]\config\external\nss\nss3.lib [moz_build]\intl\icu\target\\lib\\icuin.lib [moz_build]\intl\icu\target\lib\icuuc.lib [moz_build]\intl\icu\target\lib\icudt.lib [moz_build]\\js\src\shell\Unified_cpp_js_src_shell0.obj Expected results: ideally mozilla js engine dependencies should be resolved via one unified library. e.g. moz_js.lib
close to 36 release so this can wait until 37. doesn't block anything, just a tidy up.
Component: Untriaged → JavaScript Engine
Product: Firefox → Core
Did you build with --enabled-shared-js? If you didn't, JS is linked directly into the XUL library.
Flags: needinfo?(steve)
got this (commented out) in my mozconfig, boris. #ac_add_options --disable-shared-js so building with whatever the default is. have got it linking again now so no issue here. thought you might like it a bit more 'tidy' for 3rd party apps. that said, it's not a big deal to just inspect how js.exe binds, so I'm good doing just that. it's an FYI rather than a feature request. totally up to you whether you think this is worth refining. 'wontfix' is cool. I have a method for staying in sync.
Flags: needinfo?(steve)
don't want XUL in this case, just the js engine, but like I say, not a big deal.
Status: UNCONFIRMED → RESOLVED
Closed: 2 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.