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

Move to Gradle dependencies #11

Merged
merged 1 commit into from
May 30, 2016
Merged

Move to Gradle dependencies #11

merged 1 commit into from
May 30, 2016

Conversation

nolith
Copy link
Contributor

@nolith nolith commented Dec 3, 2015

I propose that you move to Gradle based dependency rather than a dependent
plugin. When you link in the other plugin it may cause multiple dexing issues
with other plugins.

I propose that you move to Gradle based dependency rather than a dependent
plugin. When you link in the other plugin it may cause multiple dexing issues
with other plugins.
@alexanderwallin
Copy link

I agree, <framework /> is the way to go. However, to prevent Play Services 9.x to break the plugin the lib version should be set to 8.+.

@neekfenwick
Copy link

What's going on here guys? Our project is experiencing serious pain because of the inability to have phonegap-plugin-push and this analytics plugin installed at the same time. See https://groups.google.com/forum/#!topic/phonegap/u_KG7_M4IJg for my discussion and the phonegap-plugin-push author's feedback.

@polyn0m
Copy link
Contributor

polyn0m commented May 19, 2016

+1, this pull request must be merged!

If use cordova-plugin-crosswalk-webview without this fix, build throw error:

:transformClassesWithDexForArmv7Debug
UNEXPECTED TOP-LEVEL EXCEPTION:
com.android.dex.DexIndexOverflowException: method ID not in [0, 0xffff]: 65536
at com.android.dx.merge.DexMerger$6.updateIndex(DexMerger.java:484)
at com.android.dx.merge.DexMerger$IdMerger.mergeSorted(DexMerger.java:261)
at com.android.dx.merge.DexMerger.mergeMethodIds(DexMerger.java:473)
at com.android.dx.merge.DexMerger.mergeDexes(DexMerger.java:161)
at com.android.dx.merge.DexMerger.merge(DexMerger.java:188)
at com.android.dx.command.dexer.Main.mergeLibraryDexBuffers(Main.java:504)
at com.android.dx.command.dexer.Main.runMonoDex(Main.java:334)
at com.android.dx.command.dexer.Main.run(Main.java:277)
at com.android.dx.command.dexer.Main.main(Main.java:245)
at com.android.dx.command.Main.main(Main.java:106)

FAILED

FAILURE: Build failed with an exception.

  • What went wrong:
    Execution failed for task ':transformClassesWithDexForArmv7Debug'.

    com.android.build.api.transform.TransformException: com.android.ide.common.process.ProcessException: org.gradle.process.internal.ExecException: Process 'command '/Library/Java/JavaVirtualMachines/jdk1.8.0_73.jdk/Contents/Home/bin/java'' finished with non-zero exit value 2

  • Try:
    Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

I use this fix and problem solved!

@alexanderwallin
Copy link

This has already been resolved, no?

429adea

@polyn0m
Copy link
Contributor

polyn0m commented May 20, 2016

@alexanderwallin if you use cordova-plugin-crosswalk-webview, like me, you get error when build APK, otherwise 429adea fix problem.

I think d9dba33 a better solution, because it's work always.

@miqmago
Copy link
Collaborator

miqmago commented May 25, 2016

Please, could you fix it with the newest HEAD?
As there has been some work after your commit, it's not possible to merge it.
Thanks!

@polyn0m
Copy link
Contributor

polyn0m commented May 25, 2016

@miqmago this one c6aa831? Create pull request?

@miqmago
Copy link
Collaborator

miqmago commented May 25, 2016

That's it, can you create a PR from that repo pls?

@miqmago miqmago merged commit d9dba33 into appfeel:master May 30, 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.

5 participants