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

[TIMOB-16320] Android: Integrate v7 compat library #5314

Merged
merged 10 commits into from Feb 6, 2014
Merged

[TIMOB-16320] Android: Integrate v7 compat library #5314

merged 10 commits into from Feb 6, 2014

Conversation

vishalduggal
Copy link
Contributor

This resolves TIMOB-16320 and unblocks us from continuing work on TIMOB-15021

This essentially loads v7-appcompat library as a module in the titanium SDK. Also removes a bunch of unrequired resources and puts the hooks in to build namespace specific resources.

This PR has an ugly hack to get the appcompat library to be included (see update to KrollAPIUpdater.java) which should be revisited by us when we work on TIMOB-11360.

For testing just build the SDK and run any titanium app. There should be no change in functionality. This PR does not change the build scripts to generate new themes based on the AppCompat library. That will be done as part of TIMOB-15021.

@hieupham007
Copy link
Contributor

git is ignoring empty folders. i.e: Titanium_Mobile/android/modules/appcompat/src/java. When I create that folder manually, it works fine.

@vishalduggal
Copy link
Contributor Author

Changes pushed. Ready for review

@hieupham007
Copy link
Contributor

Code reviewed and functionally tested. Request accepted

hieupham007 added a commit that referenced this pull request Feb 6, 2014
[TIMOB-16320] Android: Integrate v7 compat library
@hieupham007 hieupham007 merged commit fe80927 into tidev:master Feb 6, 2014
@donaldking
Copy link

Hi, How do i install this? I currently have 3.2.2.GA SDK but when I do custom theme for Android, it doesn't work cos android support lib v7 is not installed. Please can you advise?

@ingo
Copy link
Contributor

ingo commented Mar 20, 2014

http://builds.appcelerator.com/#master. Download the latest build. You have to be using 3.3.0

@vishalduggal vishalduggal deleted the timob-16320 branch October 27, 2014 19:16
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

5 participants