Closed Bug 249028 Opened 20 years ago Closed 20 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: 20 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: 20 years ago20 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: