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

Use Manifest Merger v2 #519

Closed
Shusshu opened this issue Nov 25, 2014 · 6 comments
Closed

Use Manifest Merger v2 #519

Shusshu opened this issue Nov 25, 2014 · 6 comments

Comments

@Shusshu
Copy link
Member

Shusshu commented Nov 25, 2014

Ticket to not forget the manifest merger v2

Have a flag to use the old manifest-merger default to the new one

Linked issue: #511

@Shusshu
Copy link
Member Author

Shusshu commented Dec 5, 2014

@Shusshu
Copy link
Member Author

Shusshu commented Dec 8, 2014

I did some basic stuff here in a new Mojo:
https://github.com/simpligility/android-maven-plugin/tree/feature-manifest-merger-v2

Manifest Merger v2 user guide: http://tools.android.com/tech-docs/new-build-system/user-guide/manifest-merger

Everything you setup in your AndroidManifest with the tag tools should work

Setup used in my apk pom:

<sourceManifestFile>${project.basedir}/src/main/AndroidManifest.xml</sourceManifestFile>
<androidManifestFile>${project.build.directory}/AndroidManifest.xml</androidManifestFile>
<manifestMerger>
        <versionName>4.2.0-CUSTOM</versionName>
        <versionCode>42</versionCode>
        <usesSdk>
            <minSdkVersion>14</minSdkVersion>
            <targetSdkVersion>21</targetSdkVersion>
        </usesSdk>
</manifestMerger>

With that setup you don't need to modify your AndroidManifest on every release and commit it...
Everything is done in the pom!

@mosabua
Copy link
Member

mosabua commented Dec 10, 2014

I think we can just remove the old one provided we have all features of the old one.

@Shusshu
Copy link
Member Author

Shusshu commented Dec 10, 2014

I don't think we'll have the same features for example the autoincrement won't be part of it I guess...

@mosabua
Copy link
Member

mosabua commented Dec 18, 2014

autoincrement is useful imho. Setting same version as in pom is essential imho..

@Shusshu
Copy link
Member Author

Shusshu commented Dec 18, 2014

"Setting same version as in pom is essential imho.." that's part of it.

AutoIncrement isn't part of it.
Since we now take the manifest from the src folder process it using values from the pom and place it in the target folder how would you autoincrement it?

@Shusshu Shusshu closed this as completed Dec 19, 2014
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

No branches or pull requests

2 participants