Closed
Bug 1135493
Opened 10 years ago
Closed 2 years ago
javascript engine unified library
Categories
(Core :: JavaScript Engine, defect)
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
Reporter | ||
Comment 1•10 years ago
|
||
close to 36 release so this can wait until 37. doesn't block anything, just a tidy up.
Reporter | ||
Updated•10 years ago
|
Component: Untriaged → JavaScript Engine
Product: Firefox → Core
Comment 2•10 years ago
|
||
Did you build with --enabled-shared-js? If you didn't, JS is linked directly into the XUL library.
Flags: needinfo?(steve)
Reporter | ||
Comment 3•10 years ago
|
||
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)
Reporter | ||
Comment 4•10 years ago
|
||
don't want XUL in this case, just the js engine, but like I say, not a big deal.
Updated•2 years ago
|
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.
Description
•