Closed
Bug 915129
Opened 11 years ago
Closed 11 years ago
Don't link in epoll_sub.c from libevent on Android or B2G.
Categories
(Core :: IPC, defect)
Tracking
()
RESOLVED
FIXED
mozilla26
People
(Reporter: jld, Assigned: jld)
References
Details
(Whiteboard: [c= p= s=2013.09.20 u=])
Attachments
(1 file)
(deleted),
patch
|
bent.mozilla
:
review+
|
Details | Diff | Splinter Review |
Android's libc has syscall stubs for epoll_{create,ctl,wait}, so we don't need libevent's replacements that call syscall(). More importantly, Android's libc has unwinding info for epoll_* but not syscall(), so this will be a very noticeable papercut when bug 810526 lands.
But we do need them when building for regular Linux, because otherwise it breaks the build, because apparently we support a relatively old glibc. (Symbol versions suggest that 2.3.2 added them.) libevent's configure script would deal with this, but we're not running it. Thus, this patch.
https://tbpl.mozilla.org/?tree=Try&rev=651fee590a65
See also bug 905521 for discussion of the bionic side of this.
Attachment #802963 -
Flags: review?(bent.mozilla)
Comment on attachment 802963 [details] [diff] [review]
rm-epoll-sub.diff
rs=me if this builds and passes tests :)
Attachment #802963 -
Flags: review?(bent.mozilla) → review+
Assignee | ||
Comment 2•11 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=9012e2011db1 has passed everything except the Android 2.2 tests which look like they have a huge backlog (and haven't even started yet), and if this change was going to fail at all it was almost certainly going to break the build, and in fact the builds all passed. So I'm feeling confident about the cost/benefit here.
For checkin to b2g-inbound.
Keywords: checkin-needed
Comment 3•11 years ago
|
||
Keywords: checkin-needed
Comment 4•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
Updated•11 years ago
|
Whiteboard: [c= p= s=2013.09.20 u=]
You need to log in
before you can comment on or make changes to this bug.
Description
•