Closed Bug 329796 Opened 19 years ago Closed 19 years ago

Can't debug XForms on Mac OS X

Categories

(Firefox Build System :: General, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jhpedemonte, Assigned: jhpedemonte)

Details

(Keywords: fixed1.8.0.4, fixed1.8.1)

Attachments

(1 file, 1 obsolete file)

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.
Attached patch patch (obsolete) (deleted) — Splinter Review
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.
Attached patch patch v2 - global change (deleted) — Splinter Review
This should be fixed in rules.mk. It would be foolish to make all users add the MOZ_DEBUG check around STRIP_XPI.
Attachment #214445 - Attachment is obsolete: true
Attachment #215294 - Flags: review?(benjamin)
Attachment #215294 - Flags: review?(benjamin) → review+
Checked in to trunk. ->FIXED
Status: NEW → RESOLVED
Closed: 19 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: approval1.8.0.3?
Attachment #215294 - Flags: approval-branch-1.8.1?(benjamin)
Attachment #215294 - Flags: approval-branch-1.8.1?(benjamin) → approval-branch-1.8.1+
Keywords: fixed1.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: approval1.8.0.3? → approval1.8.0.3+
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.
Keywords: fixed1.8.0.4
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: