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

Create sample proguard files #381

Closed
spyhunter99 opened this issue Aug 22, 2016 · 4 comments
Closed

Create sample proguard files #381

spyhunter99 opened this issue Aug 22, 2016 · 4 comments
Labels

Comments

@spyhunter99
Copy link
Collaborator

@spyhunter99 spyhunter99 commented Aug 22, 2016

primarily for

  • minified osmdroid-android
  • everything osmdroid-android (complete)
@k3b
Copy link
Contributor

@k3b k3b commented Aug 24, 2016

Helpfull for this: let gradle show dependencies to other modules:

All i found was this:

  • gradle osmdroid-forge-app:dependencies

shows this:

+--- org.mapsforge:mapsforge-map-android:0.6.0
|    +--- org.mapsforge:mapsforge-map-reader:0.6.0
|    |    \--- org.mapsforge:mapsforge-map:0.6.0
|    |         +--- org.mapsforge:mapsforge-core:0.6.0
|    |         \--- net.sf.kxml:kxml2:2.3.0
|    \--- com.caverock:androidsvg:1.2.2-beta-1

i assume that net.sf.kxml:kxml2:2.3.0 is not necessary to use mapsforge.

i assume that this could do the trick (without proguaard) but i havn-t tested it yet

  • compile('osmdroid-mapsforge:5.2') {transitive = false}
  • compile('org.mapsforge:mapsforge-map-android:0.6.0') {transitive = false}
  • compile('org.mapsforge:mapsforge-map:0.6.0') {transitive = false}
  • compile('org.mapsforge:mapsforge-core:0.6.0') {transitive = false}

However this might break when mapsforge changes it-s dependencies.

Is there somebody out there who can verify that kml-stuff is not needed?

Is there an other syntax that only excludes net.sf.kxml:kxml2without touching the other mapsforge dependencies?

@k3b k3b mentioned this issue Aug 24, 2016
9 of 9 tasks complete
@spyhunter99
Copy link
Collaborator Author

@spyhunter99 spyhunter99 commented Aug 24, 2016

There is. Here's an example

 compile('ch.acra:acra:4.7.0') {
        exclude group: 'org.json'
    }
@k3b
Copy link
Contributor

@k3b k3b commented Aug 26, 2016

kxml as a memory efficient xml praser. it has nothing to do with kml. i compiled the forge demo app without it and it seems to be working.

@spyhunter99
Copy link
Collaborator Author

@spyhunter99 spyhunter99 commented Aug 26, 2016

Forge uses xml for defining rendering settings. I think it's required. Did
your test include a clean build?

On Fri, Aug 26, 2016 at 9:25 AM, k3b notifications@github.com wrote:

kxml http://www.kxml.org/ as a memory efficient xml praser. it has
nothing to do with kml. i compiled the forge demo app without it and it
seems to be working.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#381 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB4kGSCH_dN46Vcfd8xlFmEYy62jJ8YDks5qjuk5gaJpZM4JqVyv
.

spyhunter99 added a commit that referenced this issue Aug 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants