[TIMOB-19795] Android: Update native code to work against latest LTS V8 APIs
| GitHub Issue | n/a |
|---|---|
| Type | Story |
| Priority | Critical |
| Status | Closed |
| Resolution | Fixed |
| Resolution Date | 2016-02-29T19:55:14.000+0000 |
| Affected Version/s | n/a |
| Fix Version/s | Release 6.0.0 |
| Components | Android |
| Labels | n/a |
| Reporter | Christopher Williams |
| Assignee | Christopher Williams |
| Created | 2015-10-26T14:50:25.000+0000 |
| Updated | 2017-03-21T20:46:18.000+0000 |
Description
Once the V8 library is updated we'll need to do some significant work to update our native C++ code. There have been a number of large breaking API changes in V8 that we'll need to handle.
See:
* https://strongloop.com/strongblog/node-js-v0-12-c-apis-breaking/
* https://developers.google.com/v8/embed
I have this _mostly_ compiling on the v8-update branch now: https://github.com/appcelerator/titanium_mobile/tree/v8-update Right now it's failing at the end from what appears to be a "thin" ICU library issue. I believe the ICU static library was generated as "thin" (pointing at the built *.o files) and therefore we fail at the end due to missing references. I'll need to further tweak the V8_titanium build scripts to fix that.
This is now *working* on the v8-update branch (https://github.com/appcelerator/titanium_mobile/tree/v8-update). I have the standard two-tab classic Titanium app running fine on an Android emulator using V8 4.8.271.17, which is the latest stable version of V8 for Android (committed Jan 12, 2016). This brings us up to date from V8 3.9.24.29 (released way back on May 24th, 2012! Which means we have a version of V8 which is 3 years, 7 months and 20 days newer!)
Closing ticket as fixed.