If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

support android for building tamarin shell

VERIFIED FIXED

Status

Tamarin
Build Config
VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: Dan Schaffer, Assigned: Dan Schaffer)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
have a rough cut at building the tamarin shell on android working.  also the android adb tool offers ssh like support to copy files and remote execute to the emulator or android device
a few issues:
- is not working with cross-compilation system yet
- compiling other-licenses/zlib gives many warnings, removed -Werror
- release build is not working yet
- debug gives assertions when a vm is run
(Assignee)

Comment 1

8 years ago
Created attachment 424630 [details] [diff] [review]
patch to build android
(Assignee)

Updated

8 years ago
Attachment #424630 - Flags: review?(stejohns)

Comment 2

8 years ago
Comment on attachment 424630 [details] [diff] [review]
patch to build android

Note: MMgcCustomBuild.h isn't necessary anymore; nothing anywhere should include it.
Attachment #424630 - Flags: review?(stejohns) → review+
(Assignee)

Comment 3

8 years ago
the part of the patch in other-licenses/zlib/mozzconf.h is pretty bad:
#ifdef compress
#undef compress
#endif

any better solutions?  How can we do this in our code (not in our copy of mozilla zlib)?  Is there an android specific file or an android #define?

Comment 4

8 years ago
Don't know. I vaguely recall seeing something similar in OSX-specific headers in the past.
(Assignee)

Comment 5

8 years ago
pushed in tamarin-redux 3718:37e6a906f681
(Assignee)

Updated

8 years ago
Assignee: nobody → dschaffe
(Assignee)

Updated

8 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

8 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

8 years ago
marking verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.