Closed Bug 329796 Opened 18 years ago Closed 18 years ago
Can't debug XForms on Mac OS X
Bug 309210 made it so debug symbols are stripped from the xforms lib when packaging it in the XPI. However, it does the stripping for debug builds also, so I can't set breakpoints when trying to debug XForms.
Don't strip symbols if MOZ_DEBUG is defined.
Attachment #214445 - Flags: review?(doronr)
Comment on attachment 214445 [details] [diff] [review] patch not really a build config bug, but a xforms bug :)
Attachment #214445 - Flags: review?(doronr) → review+
Actually, I'm wondering if we shouldn't wrap the rule in rules.mk with "ifndef MOZ_DEBUG", since I don't really see a reason why we would want to strip if we are building debug. If not, this forces everyone who defines STRIP_XPI to also wrap it with the ifdef.
Same goes for Linux.
This should be fixed in rules.mk. It would be foolish to make all users add the MOZ_DEBUG check around STRIP_XPI.
Checked in to trunk. ->FIXED
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Comment on attachment 215294 [details] [diff] [review] patch v2 - global change Asking for approval for both branches. Without this patch, we cannot debug XForms XPI on Mac or Linux.
Attachment #215294 - Flags: approval-branch-1.8.1?(benjamin) → approval-branch-1.8.1+
Comment on attachment 215294 [details] [diff] [review] patch v2 - global change approved for 1.8.0 branch, a=dveditz for drivers
Attachment #215294 - Flags: approval18.104.22.168? → approval22.214.171.124+
Code freeze for this release is Monday May 1, at which point approval will be withdrawn from this bug if it's not checked in.
You need to log in before you can comment on or make changes to this bug.