Closed
Bug 249028
Opened 20 years ago
Closed 20 years ago
Make XPCOM_DEBUG_BREAK=trap work on OS X
Categories
(Core :: XPCOM, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.8alpha4
People
(Reporter: peterv, Assigned: cls)
References
Details
Attachments
(1 file, 4 obsolete files)
587 bytes,
patch
|
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
Reporter | ||
Comment 1•20 years ago
|
||
Reporter | ||
Updated•20 years ago
|
Attachment #151939 -
Flags: superreview?(dougt)
Attachment #151939 -
Flags: review?(bryner)
Updated•20 years ago
|
Attachment #151939 -
Flags: review?(bryner) → review+
Comment 2•20 years ago
|
||
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+
Reporter | ||
Updated•20 years ago
|
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Comment 3•20 years ago
|
||
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
Comment 4•20 years ago
|
||
Should I file a separate bug?
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 →
Attachment #154070 -
Attachment description: don't require MacTypes.h →
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)
Attachment #156750 -
Flags: superreview?(dougt) → superreview?(darin)
Updated•20 years ago
|
Attachment #156750 -
Flags: superreview?(darin) → superreview+
Attachment #156750 -
Flags: review?(bryner)
Patch has been checked in.
Status: NEW → RESOLVED
Closed: 20 years ago → 20 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.8alpha4
Assignee | ||
Comment 10•19 years ago
|
||
*** 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.
Description
•