Skip to content
This repository has been archived by the owner on Aug 25, 2020. It is now read-only.

removed attributes from Manifest #41

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

removed attributes from Manifest #41

wants to merge 1 commit into from

Conversation

metinkale38
Copy link

these attributes should belong to the application project, not to the library project
They cause problems in applications project using that library, e.g.:

Attribute application@label value=(@string/appName) from [:features:base] AndroidManifest.xml:51:9-40
is also present at [us.feras.mdv:markdownview:1.1.0] AndroidManifest.xml:13:9-41 value=(@string/app_name).
Suggestion: add 'tools:replace="android:label"' to element at AndroidManifest.xml:6:5-157:19 to override.

It might be easily fixable with the tools:replace Attribute, but thats more a workaround than a real fix...

these attributes should belong to the application project, not to the library project
They cause problems in applications project using that library, e.g.:

Attribute application@label value=(@string/appName) from [:features:base] AndroidManifest.xml:51:9-40
	is also present at [us.feras.mdv:markdownview:1.1.0] AndroidManifest.xml:13:9-41 value=(@string/app_name).
	Suggestion: add 'tools:replace="android:label"' to <application> element at AndroidManifest.xml:6:5-157:19 to override.

It might be easily fixable with the tools:replace Attribute, but thats more a workaround than a real fix...
Copy link

@BoxResin BoxResin left a comment

Choose a reason for hiding this comment

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

Nice.

@andschdk
Copy link

andschdk commented Dec 6, 2019

Would be nice to have this merged so you shouldn't add tools:replace="android:allowBackup" to your apps manifest.

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.

None yet

3 participants