Closed
Bug 351472
Opened 18 years ago
Closed 18 years ago
Building trunk with GCC 3.2.2 on OS/2 fails in java_xpcom.c
Categories
(Core Graveyard :: Java to XPCOM Bridge, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Assigned: mozilla)
References
Details
(Keywords: fixed1.8.1)
Attachments
(1 file)
905 bytes,
patch
|
jhpedemonte
:
review+
mtschrep
:
approval1.8.1+
|
Details | Diff | Splinter Review |
Following the checkin from bug 333618 java_xpcom.c does not compile any more on OS/2 with GCC 3.2.2: I:/mozilla/xpcom/typelib/xpidl/xpidl_java.c: In function `interface_declaration': I:/mozilla/xpcom/typelib/xpidl/xpidl_java.c:253: `PATH_MAX' undeclared (first use in this function) I:/mozilla/xpcom/typelib/xpidl/xpidl_java.c:253: (Each undeclared identifier is reported only once I:/mozilla/xpcom/typelib/xpidl/xpidl_java.c:253: for each function it appears in.) It seems that GCC 3.2.2 does not define MAX_PATH by default.
Assignee | ||
Comment 1•18 years ago
|
||
This should fix it. But I wonder if the same patch is needed for extensions/java/xpcom/tools/xpidl/xpidl_java.c.
Assignee: jhpedemonte → mozilla
Status: NEW → ASSIGNED
Attachment #236865 -
Flags: review?(jhpedemonte)
Comment 2•18 years ago
|
||
Comment on attachment 236865 [details] [diff] [review] the fix Looks good. None of the code in extensions/java/xpcom/tools is used any more after bug 333618, so don't worry about it.
Attachment #236865 -
Flags: review?(jhpedemonte) → review+
Assignee | ||
Comment 3•18 years ago
|
||
Checked into trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 4•18 years ago
|
||
Comment on attachment 236865 [details] [diff] [review] the fix Not that the respective patch for bug 333618 went into MOZILLA_1_8_BRANCH we should get this in, too.
Attachment #236865 -
Flags: approval1.8.1?
Comment 5•18 years ago
|
||
Comment on attachment 236865 [details] [diff] [review] the fix a=schrep for os2 build fix.
Attachment #236865 -
Flags: approval1.8.1? → approval1.8.1+
Assignee | ||
Updated•18 years ago
|
Keywords: fixed1.8.1
Updated•10 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•