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 leper)

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)

Fix-debug.diff (352 bytes ) - added by Josh 10 years ago.

Download all attachments as: .zip

Change History (11)

by Josh, 10 years ago

Attachment: Fix-debug.diff added

comment:1 by Josh, 10 years ago

Description: modified (diff)

Whoops! That patch only seems to fix the build, the linker still fails. The errors are here: http://pastebin.com/S7vs24NV.

comment:2 by Josh, 10 years ago

Cc: Yves added

comment:3 by Yves, 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 historic_bruno, 10 years ago

Priority: Release BlockerShould 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 Josh, 10 years ago

Owner: Josh 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 Yves, 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:7 by agentx, 10 years ago

probably relevant #2693

Last edited 10 years ago by historic_bruno (previous) (diff)

comment:8 by historic_bruno, 10 years ago

Milestone: Alpha 17Alpha 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 Simon Georges, 9 years ago

Josh, is there a change on this issue with the recent update of Spider Monkey ?

comment:10 by leper, 9 years ago

Description: modified (diff)
Milestone: Alpha 18Backlog

Rewrote the ticket since debug builds should use debug libraries and fail otherwise. Also moving to backlog.

Note: See TracTickets for help on using tickets.