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

Update android manifest for module api version change #11

Closed
wants to merge 1 commit into from

Conversation

sgtcoolguy
Copy link

@sgtcoolguy sgtcoolguy commented Jul 25, 2016

Hey guys, Titanium 6 will introduce a breaking change to Android modules, as a result of upgrading the V8 engine used. Going from 5.x to 6.x, the module apiversion will be bumped from 2 to 3, "armeabi" ABI will be removed and modules will need to be rebuilt against a 6.0.0+ version of the SDK.

These changes are basically all that are needed in terms of the your module. Then you'll need to grab a 6.0.0/master build of the SDK:
appc ti sdk install -b master

And point your build.properties at that version, then just re-run ant to build a new android module zip.

@rlepinski
Copy link
Collaborator

@sgtcoolguy Thanks! Should we wait to merge/release until 6.0 is out?

Also, could you fill out a contribution agreement? I know its a small change but we require one for every commit. http://docs.urbanairship.com/contribution-agreement.html

@sgtcoolguy
Copy link
Author

Just signed the CLA.

It'd be good to build it against 6.0.0 and test it out locally and perhaps offer it as pre-release/beta to any customers who may be trying the 6.x builds of Titanium SDK.

It'll unfortunately end up in an awkward state eventually where 6+ requires modules built against 6+ and < 6, against < 6. So there may be an interim period where you may want to maintain two branches/release for your users?

@rlepinski
Copy link
Collaborator

@sgtcoolguy Thanks! We will try to do a release this week or the next.

@hcrowell
Copy link
Contributor

@sgtcoolguy We just updated to Titanium 6. Thanks for the heads up!

@hcrowell hcrowell closed this Nov 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants