Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

Update Manifest finder for apk splits#2065

Merged
WonderCsabo merged 2 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
dodgex:2034_update_manifest_finder_to_support_splitsCopy head branch name to clipboard
Oct 31, 2017
Merged

Update Manifest finder for apk splits#2065
WonderCsabo merged 2 commits into
androidannotations:developandroidannotations/androidannotations:developfrom
dodgex:2034_update_manifest_finder_to_support_splitsCopy head branch name to clipboard

Conversation

@dodgex

@dodgex dodgex commented Oct 30, 2017

Copy link
Copy Markdown
Member

This PR is aimed to fix #2034 and allow android annotations to work in projects that use apk splits.

@dodgex dodgex requested a review from WonderCsabo October 30, 2017 22:56

@WonderCsabo WonderCsabo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to be good, but we should definitely have some tests for this.

@dodgex

dodgex commented Oct 31, 2017

Copy link
Copy Markdown
Member Author

I extended the existing tests with paths for split builds

@WonderCsabo WonderCsabo merged commit f078148 into androidannotations:develop Oct 31, 2017
@WonderCsabo WonderCsabo added this to the 4.4 milestone Oct 31, 2017
@soster

soster commented Nov 18, 2017

Copy link
Copy Markdown

I have the same problem using AA 4.4, shouldn't it be fixed in that version?
Using ABI split, and getting this:
error: Could not find the AndroidManifest.xml file in specified path : null

@soster

soster commented Nov 18, 2017

Copy link
Copy Markdown

Nevermind, I still had a custom androidManifestFile configuration, after deleting it it worked...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Could not find AndroidManifest.xml file using generation folder

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.