Closed
Bug 249028
Opened 21 years ago
Closed 21 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•21 years ago
|
||
| Reporter | ||
Updated•21 years ago
|
Attachment #151939 -
Flags: superreview?(dougt)
Attachment #151939 -
Flags: review?(bryner)
Updated•21 years ago
|
Attachment #151939 -
Flags: review?(bryner) → review+
Comment 2•21 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•21 years ago
|
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment 3•21 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•21 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•21 years ago
|
Attachment #156750 -
Flags: superreview?(darin) → superreview+
Attachment #156750 -
Flags: review?(bryner)
Patch has been checked in.
Status: NEW → RESOLVED
Closed: 21 years ago → 21 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.8alpha4
| Assignee | ||
Comment 10•20 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
•