Closed
Bug 859580
Opened 12 years ago
Closed 12 years ago
Include asm/sigcontext.h for older Bionic versions which do not include this from signal.h
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: dougc, Unassigned)
References
Details
Attachments
(1 file)
1.27 KB,
patch
|
mjrosenb
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:20.0) Gecko/20100101 Firefox/20.0
Build ID: 20130401133302
Steps to reproduce:
Compiling asm.js support for b2g ARM.
Actual results:
Missing definition of sigcontext.
Expected results:
Needs to include asm/sigcontext.h for the older version of Bionic used by b2g.
Reporter | ||
Comment 1•12 years ago
|
||
Attachment #734918 -
Flags: review?(mrosenberg)
Reporter | ||
Updated•12 years ago
|
Component: Untriaged → JavaScript Engine
OS: Linux → Android
Product: Firefox → Core
Hardware: x86_64 → ARM
Comment 2•12 years ago
|
||
Comment on attachment 734918 [details] [diff] [review]
Include asm/sigcontext.h for older Bionic versions which do not include this from signal.h
Review of attachment 734918 [details] [diff] [review]:
-----------------------------------------------------------------
hey, this was blocking me from landing, I'm just going to grab this patch and land it with the rest of the asm.js/arm code.
Attachment #734918 -
Flags: review?(mrosenberg) → review+
Reporter | ||
Comment 3•12 years ago
|
||
The patch landed as part of bug 840285 and is now resolved, thanks.
Status: UNCONFIRMED → RESOLVED
Closed: 12 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•