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

Package changing and including proguard configuration to library #102

Conversation

emartynov
Copy link
Contributor

Goal

Avoid manual configuration for obfuscation. Android gradle plugin takes care about it now.

I also updated maven version plugin and fix one crash related to configuration.

@johncarl81 please review

@emartynov
Copy link
Contributor Author

Obfuscation is failing on jdk8 build. I'm looking for solution

@emartynov emartynov force-pushed the android-library-proguard-including branch from 7f120b2 to cb1c26f Compare June 5, 2015 12:40
@emartynov
Copy link
Contributor Author

It builds but if I try to deploy example app - it fails on run since class is obfuscated wrongly. I've tried also to use gradle instead of maven to build example app and got some build errors.

It is a big shame that I can not spend appropriate amount of time to investigate the problem and finally find proper solution.

John, if no one won't help on this I would ask you to close PR.

@johncarl81
Copy link
Owner

Eugen,

I'll take a look at this shortly. Maybe fresh eyes will help.
On Jul 11, 2015 3:41 AM, "Eugen Martynov" notifications@github.com wrote:

It builds but if I try to deploy example app - it fails on run since class
is obfuscated wrongly. I've tried also to use gradle instead of maven to
build example app and got some build errors.

It is a big shame that I can not spend appropriate amount of time to
investigate the problem and finally find proper solution.

John, if no one won't help on this I would ask you to close PR.


Reply to this email directly or view it on GitHub
#102 (comment).

@johncarl81 johncarl81 closed this Sep 27, 2015
@johncarl81 johncarl81 mentioned this pull request Dec 27, 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

2 participants