PomepuyN / discreet-app-rate

A lightweight non intrusive app rate reminder for Android
http://nicolaspomepuy.fr/non-intrusive-app-rate
Apache License 2.0
464 stars 51 forks source link

Manifest merging failed #28

Closed Thangiee closed 10 years ago

Thangiee commented 10 years ago

When i update from version 1.0.5 to 2.0.2, i am getting this:

[warn] [.../com.android.support-appcompat-v7-21.0.0-rc1/AndroidManifest.xml:3] Failed to parse <uses-sdk minSdkVersion='L'>: must be an integer number or codename.
[warn] [.../com.android.support-appcompat-v7-21.0.0-rc1/AndroidManifest.xml:3] Failed to parse <uses-sdk targetSdkVersion='L'>: must be an integer number or codename.
[warn] [.../com.android.support-support-v4-21.0.0-rc1/AndroidManifest.xml:3] Failed to parse <uses-sdk minSdkVersion='L'>: must be an integer number or codename.
[warn] [.../com.android.support-support-v4-21.0.0-rc1/AndroidManifest.xml:3] Failed to parse <uses-sdk targetSdkVersion='L'>: must be an integer number or codename.
[info] Performing full resource merge
java.lang.RuntimeException: Manifest merging failed. See console for more info.
        at com.android.builder.core.AndroidBuilder.doMerge(AndroidBuilder.java:920)
        at com.android.builder.core.AndroidBuilder.mergeLibraryManifests(AndroidBuilder.java:906)
        at com.android.builder.core.AndroidBuilder.processManifest(AndroidBuilder.java:612)
        at android.Tasks$$anonfun$58.apply(tasks.scala:912)
        at android.Tasks$$anonfun$58.apply(tasks.scala:899)
        at scala.Function9$$anonfun$tupled$1.apply(Function9.scala:35)
        at scala.Function9$$anonfun$tupled$1.apply(Function9.scala:34)
        at scala.Function1$$anonfun$compose$1.apply(Function1.scala:47)
        at sbt.$tilde$greater$$anonfun$$u2219$1.apply(TypeFunctions.scala:42)
        at sbt.std.Transform$$anon$4.work(System.scala:64)
        at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:237)
        at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:237)
        at sbt.ErrorHandling$.wideConvert(ErrorHandling.scala:18)
        at sbt.Execute.work(Execute.scala:244)
        at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:237)
        at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:237)
        at sbt.ConcurrentRestrictions$$anon$4$$anonfun$1.apply(ConcurrentRestrictions.scala:160)
        at sbt.CompletionService$$anon$2.call(CompletionService.scala:30)
        at java.util.concurrent.FutureTask.run(FutureTask.java:262)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
        at java.util.concurrent.FutureTask.run(FutureTask.java:262)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
        at java.lang.Thread.run(Thread.java:745)
[error] (android:processManifest) Manifest merging failed. See console for more info.

It seems like this library is pulling in the latest android support revisions.

Could it be that the "+" is causing it to do that?

dependencies {
compile 'com.android.support:appcompat-v7:+'
}
PomepuyN commented 10 years ago

Could you please copy/paste me the dependancy section of your build.gradle file?

Thangiee commented 10 years ago

I'm using scala and this sbt plugin so i'm not sure if that could be affecting this in some way. But anyhow, here are the dependencies from my build.sbt .

libraryDependencies ++= Seq(
  "org.scaloid" %% "scaloid" % "3.4-10",
  "com.android.support" % "support-v13" % "19.+",
  "com.android.support" % "support-v4" % "19.+",
  "com.android.support" % "appcompat-v7" % "19.+",
  "com.google.code.gson" % "gson" % "2.2.4",
  "com.github.amigold.fundapter2" % "library" % "1.01",
  "de.keyboardsurfer.android.widget" % "crouton" % "1.8.4",
  "de.greenrobot" % "eventbus" % "2.2.1",
  "com.github.nscala-time" %% "nscala-time" % "1.2.0",
  "org.jsoup" % "jsoup" % "1.7.3",
  "ch.acra" % "acra" % "4.5.0",
  aar("com.github.gabrielemariotti.changeloglib" % "library" % "1.5.1"),
  aar("de.psdev.licensesdialog" % "licensesdialog" % "1.5.0"),
  aar("com.google.android.gms" % "play-services" % "5.0.+"),
  aar("com.anjlab.android.iab.v3" % "library" % "1.0.+"),
  aar("com.github.dmytrodanylyk.circular-progress-button" % "library" % "1.1.0"),
  aar("net.simonvt.menudrawer" % "menudrawer" % "3.0.6"),
  aar("com.github.gabrielemariotti.cards" % "library-extra" % "1.8.0"),
  aar("info.hoang8f" % "android-segmented" % "1.0.2"),
  aar("com.makeramen" % "roundedimageview" % "1.3.0"),
  aar("com.pixplicity.easyprefs" % "library" % "1.3"),
  aar("fr.nicolaspomepuy" % "discreetapprate" % "1.0.5"),
  aar("com.github.johnkil.android-progressfragment" % "progressfragment-native" % "1.4.0"),
  aar("com.astuetz" % "pagerslidingtabstrip" % "1.0.1"))
PomepuyN commented 10 years ago

If you remove the line corresponding to my lib, the manifest merger issue is gone?

Thangiee commented 10 years ago

Yes that is correct. If i remove it or use v1.0.5, it works. However, i just ran into a similar issue with another library. Maybe the issue is from my side? So i'll stick to 1.0.5 for the moment. Also, feel free to close the issue for now.

Thanks for your time.