Closed
Bug 654056
Opened 13 years ago
Closed 13 years ago
Build IPC support for mips, hppa, ia64, s390(x), powerpc64 and sparc
Categories
(Core :: IPC, defect)
Tracking
()
RESOLVED
FIXED
mozilla10
People
(Reporter: glandium, Assigned: glandium)
References
Details
(Whiteboard: [inbound])
Attachments
(1 file, 3 obsolete files)
|
2.54 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
The patch I'm attaching makes things build. ia64 has a broken jsengine because of the new jsval, I haven't tested hppa, and I know for a fact that it doesn't actually work on mips and s390, so it's a WIP. Note that on s390, size_t is defined as unsigned long, which explains the ipc_message_utils.h change.
| Assignee | ||
Updated•13 years ago
|
Attachment #529427 -
Attachment filename: diff → bug654056
| Assignee | ||
Comment 2•13 years ago
|
||
| Assignee | ||
Updated•13 years ago
|
Attachment #529427 -
Attachment is obsolete: true
This patch makes the work with the resource files endian-independent. It builds fine on x86_64 and s390x, but I don't know how to test it or whether it's even required besides the fact that build fails on big endian arch.
| Assignee | ||
Comment 4•13 years ago
|
||
(In reply to comment #3) > Created attachment 543098 [details] [diff] [review] [review] > fix little-endian assumption in data_pack.cc > > This patch makes the work with the resource files endian-independent. It > builds fine on x86_64 and s390x, but I don't know how to test it or whether > it's even required besides the fact that build fails on big endian arch. That file is not built anymore, as it was not used at all. See bug 643112.
| Assignee | ||
Updated•13 years ago
|
Attachment #543098 -
Attachment is obsolete: true
| Assignee | ||
Updated•13 years ago
|
OS: Linux → Mac OS X
Summary: Add IPC support for mips, hppa, ia64, s390 and sparc → Build IPC support for mips, hppa, ia64, s390(x), powerpc64 and sparc
Target Milestone: --- → mozilla9
Version: Trunk → Other Branch
| Assignee | ||
Updated•13 years ago
|
Target Milestone: mozilla9 → ---
| Assignee | ||
Comment 5•13 years ago
|
||
Let's already get this in the tree and file separate bugs for runtime issues
Attachment #563704 -
Flags: review?(benjamin)
| Assignee | ||
Updated•13 years ago
|
Attachment #543088 -
Attachment is obsolete: true
| Assignee | ||
Updated•13 years ago
|
Assignee: nobody → mh+mozilla
OS: Mac OS X → Linux
Version: Other Branch → Trunk
Updated•13 years ago
|
Attachment #563704 -
Flags: review?(benjamin) → review+
| Assignee | ||
Comment 6•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ed7e00a14cb9
Whiteboard: [inbound]
| Assignee | ||
Comment 7•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/c7134426674a
Comment 8•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/ed7e00a14cb9 https://hg.mozilla.org/mozilla-central/rev/c7134426674a
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in
before you can comment on or make changes to this bug.
Description
•