Opened 10 years ago
Last modified 9 years ago
#2555 new defect
Debug build broken when using system spidermonkey
Reported by: | Josh | Owned by: | |
---|---|---|---|
Priority: | Should Have | Milestone: | Backlog |
Component: | Build & Packages | Keywords: | |
Cc: | Yves | Patch: |
Description (last modified by )
Since system provided SpiderMonkey (at least on Ubuntu) does not provide a debug build we should fail with a clear error message when trying to perform a debug build with --with-system-mozjs*
if no debug library is present.
Possibly providing an option to create a debug build against SpiderMonkey release builds could be added, but as debug builds should provide as much information as possible this might not be useful.
Attachments (1)
Change History (11)
by , 10 years ago
Attachment: | Fix-debug.diff added |
---|
comment:1 by , 10 years ago
Description: | modified (diff) |
---|
comment:2 by , 10 years ago
Cc: | added |
---|
comment:3 by , 10 years ago
Some additional information here (starting around 12:19): http://irclogs.wildfiregames.com/2014-05-14-QuakeNet-%230ad-dev.log
comment:4 by , 10 years ago
Priority: | Release Blocker → Should Have |
---|
Not really a release blocker, almost nobody uses debug builds... but thinking about it, this can't be much different than the past, Linux packagers don't typically provide debug builds do they?
comment:5 by , 10 years ago
Owner: | removed |
---|
It's not much different than it used to be, but the debug build no longer works with system spidermonkey libraries which is a regression.
comment:6 by , 10 years ago
We should probably link to the release binaries for debug builds. Having a system provided debug build does not make much sense because there are many different options that can be enabled for debugging different aspects of the Javascript engine or scripts. The only downside is that we have some useful debugging functionality enabled by default for custom debug builds and programmers could not be aware that they get better debugging functionality if they use the custom build instead of the system library.
comment:8 by , 10 years ago
Milestone: | Alpha 17 → Alpha 18 |
---|
This isn't really a release issue, since nobody releases debug builds, so I'm pushing to A18. We can take care of it then.
comment:9 by , 9 years ago
Josh, is there a change on this issue with the recent update of Spider Monkey ?
comment:10 by , 9 years ago
Description: | modified (diff) |
---|---|
Milestone: | Alpha 18 → Backlog |
Rewrote the ticket since debug builds should use debug libraries and fail otherwise. Also moving to backlog.
Whoops! That patch only seems to fix the build, the linker still fails. The errors are here: http://pastebin.com/S7vs24NV.