Closed Bug 249028 Opened 21 years ago Closed 21 years ago

Make XPCOM_DEBUG_BREAK=trap work on OS X

Categories

(Core :: XPCOM, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.8alpha4

People

(Reporter: peterv, Assigned: cls)

References

Details

Attachments

(1 file, 4 obsolete files)

Attached patch Support XPCOM_DEBUG_BREAK=trap (obsolete) — Splinter Review
Attachment #151939 - Flags: superreview?(dougt)
Attachment #151939 - Flags: review?(bryner)
Attachment #151939 - Flags: review?(bryner) → review+
Comment on attachment 151939 [details] [diff] [review] Support XPCOM_DEBUG_BREAK=trap looks fine to me. peterv, can you check this in?
Attachment #151939 - Flags: superreview?(dougt) → superreview+
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
I'm trying to do a GTK build on Mac OS X, and I get this error: make[4]: Leaving directory `/Users/kev/MozillaSource/gtkfftrunk/mozilla/xpcom/glue' make[4]: Entering directory `/Users/kev/MozillaSource/gtkfftrunk/mozilla/xpcom/base' nsDebugImpl.cpp c++ -o nsDebugImpl.o -c -DOSTYPE=\"Darwin7.4.0\" -DOSARCH=\"Darwin\" -D_IMPL_NS_COM -I../../dist/include/string -I../../dist/include/xpcom -I../../dist/include -I/Users/kev/MozillaSource/gtkfftrunk/mozilla/dist/include/nspr -I/usr/X11R6/include -fPIC -I/usr/X11R6/include -fno-rtti -fno-exceptions -Wall -Wconversion -Wpointer-arith -Wcast-align -Woverloaded-virtual -Wsynth -Wno-ctor-dtor-privacy -Wno-non-virtual-dtor -Wno-long-long -fpascal-strings -no-cpp-precomp -fno-common -fshort-wchar -pipe -DNDEBUG -DTRIMMED -O -I/usr/X11R6/include -DMOZILLA_CLIENT -include ../../mozilla-config.h -Wp,-MD,.deps/nsDebugImpl.pp nsDebugImpl.cpp nsDebugImpl.cpp:69:24: MacTypes.h: No such file or directory nsDebugImpl.cpp: In member function `virtual nsresult nsDebugImpl::Break(const char*, int)': nsDebugImpl.cpp:352: error: `Debugger' undeclared (first use this function) nsDebugImpl.cpp:352: error: (Each undeclared identifier is reported only once for each function it appears in.) make[4]: *** [nsDebugImpl.o] Error 1 make[4]: Leaving directory `/Users/kev/MozillaSource/gtkfftrunk/mozilla/xpcom/base' make[3]: *** [libs] Error 2 make[3]: Leaving directory `/Users/kev/MozillaSource/gtkfftrunk/mozilla/xpcom' make[2]: *** [libs] Error 2 make[2]: Leaving directory `/Users/kev/MozillaSource/gtkfftrunk/mozilla' make[1]: *** [alldep] Error 2 make[1]: Leaving directory `/Users/kev/MozillaSource/gtkfftrunk/mozilla' make: *** [alldep] Error 2 make: Leaving directory `/Users/kev/MozillaSource/gtkfftrunk/mozilla' If I take out the three lines added by the patch, the build completes
Should I file a separate bug?
Attached patch (obsolete) — Splinter Review
Assignee: peterv → timeless
Attachment #151939 - Attachment is obsolete: true
Status: RESOLVED → ASSIGNED
Resolution: FIXED → ---
Attachment #154069 - Flags: review?(dougt)
Attachment #154069 - Attachment is obsolete: true
Attachment #154069 - Attachment description: don't require MacTypes.h →
Attached patch (obsolete) — Splinter Review
Attachment #154070 - Attachment description: don't require MacTypes.h →
Attached patch don't require MacTypes.h (obsolete) — Splinter Review
Attachment #154070 - Attachment is obsolete: true
Attachment #154071 - Flags: review?(dougt)
Attachment #154071 - Flags: review?(dougt) → review?(sfraser)
Debugger() comes from CoreServices which we'll only link against when building against the carbon or cocoa toolkits.
Attachment #154071 - Attachment is obsolete: true
Attachment #154069 - Flags: review?(dougt)
Attachment #154071 - Flags: review?(sfraser)
Attachment #156750 - Flags: superreview?(dougt)
Attachment #156750 - Flags: review?(bryner)
Assignee: timeless → cls
Status: ASSIGNED → NEW
Attachment #156750 - Flags: superreview?(dougt) → superreview?(darin)
Attachment #156750 - Flags: superreview?(darin) → superreview+
Attachment #156750 - Flags: review?(bryner)
Patch has been checked in.
Status: NEW → RESOLVED
Closed: 21 years ago21 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.8alpha4
*** Bug 257773 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: