Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Small changes to compile for Android #5886

Closed
vicuna opened this Issue Jan 12, 2013 · 3 comments

Comments

Projects
None yet
1 participant
@vicuna
Copy link
Collaborator

vicuna commented Jan 12, 2013

Original bug ID: 5886
Reporter: vouillon
Assigned to: meurer
Status: closed (set by @xavierleroy on 2016-12-07T10:36:50Z)
Resolution: fixed
Priority: normal
Severity: feature
Version: 4.00.1
Fixed in version: 4.01.0+dev
Category: runtime system and C interface
Related to: #6617
Monitored by: meurer

Bug description

The following patch makes it possible to compile the native runtime and the unix library for Android.

File attachments

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

vicuna commented Jan 12, 2013

Comment author: meurer

Someone send me a similar patch to get the arm port to work on Android, but I haven't had the time to look into this until now. I was esp. confused by the lack of Unix signals? Which version of the NDK did you use?

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

vicuna commented Jan 12, 2013

Comment author: vouillon

I'm using the Android NDK, Revision 8d.

The issue is that the type ucontext_t is not defined (there is no sys/ucontext.h include file). This is planned to be added in the future:
http://code.google.com/p/android/issues/detail?id=34784

I have addded a better patch which includes a definition of the type.

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

vicuna commented Jan 13, 2013

Comment author: meurer

Patch looks good, and seems to work fine. Committed to trunk with revision 13225.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.