Closed Bug 494025 Opened 16 years ago Closed 16 years ago

electrolysis: Compile initial work on Linux and Mac

Categories

(Core :: XPCOM, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: cjones, Assigned: cjones)

References

Details

Attachments

(3 files, 5 obsolete files)

We're not really tracking issues here; I'm just using this as a TODO list and patch backup.
Attached patch backup (obsolete) — Splinter Review
Sorry, this patch didn't "take" to bugzilla first time around.
Assignee: nobody → jones.chris.g
Attached file my mozconfig for building above (obsolete) —
Unfortunately had to enable RTTI, due to config/system-headers dependency on buggy g++ header (tr1/functional).
Attached patch backup v2, builds more stuff (obsolete) — Splinter Review
Remaining is chasing down link errors in TestIPCShell. Hopefully I can do that tonight. Ben T., this includes a Makefile.in for libevent. Everything in this patch depends on libevent living in ipc/chromium/third_party/libevent which is, not incidentally, where it lives in the chromium source tree. I had to patch libevent slightly to build, but it was trivial stuff. I'll see if I can extract a patch and post that here, too.
Attachment #378677 - Attachment is obsolete: true
Attached patch libevent patchSplinter Review
There's a lot of unnecessary garbage in here relating to libevent's Makefile.in being replaced by ours. I decided to leave all the garbage; I don't know the best way to manage the two. I'd vote to just overwriting it with our own. Another way may be with a |ifdef CHROMIUM_MOZILLA_BUILD ... else ...| hack.
Attached patch backup v3, compiles! (obsolete) — Splinter Review
Easiest like error EVAR --- just two libraries listed out of the order that ld wants them.
Attachment #378791 - Attachment is obsolete: true
Attached patch backup v4, rebased (obsolete) — Splinter Review
Compiles on my machine, but probably won't apply cleanly to the main repository. Looks like mq and --rebase screwed up my local copy of libevent. Ben T., we'll probably need to fix this junk by hand when you have the time.
Attachment #378796 - Attachment is obsolete: true
Blocks: 494129
Summary: Linux port of initial Electrolysis work → electrolysis: Compile initial work on Linux
I suppose it would have been too much to hope for the dashed thing to run, also. I'd prefer to track this work in another bug. Leaving this one open until we resolve the merge/import issues. For the record, the current error on starting TestIPCShell is: WARNING: Couldn't register mComponentsDir: file /home/cjones/mozilla/electrolysis/xpcom/components/nsComponentManager.cpp, line 3328 ###!!! ASSERTION: This is not supposed to fail!: 'Error', file /home/cjones/mozilla/electrolysis/js/src/xpconnect/src/nsXPConnect.cpp, line 1039 ###!!! ASSERTION: Failed to initialize nsScriptSecurityManager: 'NS_SUCCEEDED(rv)', file /home/cjones/mozilla/electrolysis/caps/src/nsScriptSecurityManager.cpp, line 3298 WARNING: Cannot create startup observer : service,@mozilla.org/scriptsecuritymanager;1: file /home/cjones/mozilla/electrolysis/embedding/components/appstartup/src/nsAppStartupNotifier.cpp, line 113 [0520/232917:FATAL:/home/cjones/mozilla/electrolysis/ipc/chromium/base/message_loop.h(460)] Check failed: MessageLoop::TYPE_IO == loop->type() (2 vs. 1) [0520/232917:FATAL:/home/cjones/mozilla/electrolysis/ipc/chromium/base/message_loop.h(460)] Check failed: MessageLoop::TYPE_IO == loop->type() (2 vs. 1) Backtrace: ./TestIPCShell [0x8100ae4] ./TestIPCShell [0x80d4e7c] ./TestIPCShell [0x80c6911] ./TestIPCShell [0x80c1d5e] ./TestIPCShell [0x80c1dd4] ./TestIPCShell [0x80ac016] ./TestIPCShell [0x80a8748] ./TestIPCShell [0x80a7d80] /lib/tls/i686/cmov/libc.so.6(__libc_start_main+0xe5) [0xb6309685] ./TestIPCShell [0x80a7c21]
Attached file mozconfig backup
Attachment #378679 - Attachment is obsolete: true
Attached patch backup v5Splinter Review
Now compatible with bent's import of libevent. Doesn't quite build due a tiers issue with libchromiumthirdparty, but close enough for government work.
Attachment #378804 - Attachment is obsolete: true
(Ben Turner) pushed ba5e6b90f6fb Ben Turner - Linux and mac, oh my
Summary: electrolysis: Compile initial work on Linux → electrolysis: Compile initial work on Linux and Mac
Pushed 21f1a7caf8a6 Chris Jones - bugs 494129, 494025: Linux (and probably OS X) at parity with Windows. fix (i) stray carriage returns; (ii) link with -lrt on linux; (iii) implement UTF8ToUTF16 (hacky) converter for linux; (iv) remap child end of socketpair to well-known fd on POSIX
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: