swagger-api / swagger-core

Examples and server integrations for generating the Swagger API Specification, which enables easy access to your REST API
http://swagger.io
Apache License 2.0
7.36k stars 2.17k forks source link

Broken mockito-all dependency in swagger-play 1.3-SNAPSHOT #348

Closed dragisak closed 10 years ago

dragisak commented 10 years ago
  "org.mockito" % "mockito-all" % "1.9.0" % "test" withSources() withJavadoc())

Produces weird dependency in deployed POM file.

See https://oss.sonatype.org/content/repositories/snapshots/com/wordnik/swagger-play2_2.10/1.3-SNAPSHOT/swagger-play2_2.10-1.3-SNAPSHOT.pom

sbt tries to download file with .doc or .src extension. You get error:

  [trace] Stack trace suppressed: run last *:update for the full output.
  [error] (*:update) sbt.ResolveException: download failed: org.mockito#mockito-all;1.9.0!mockito-all.src
  [error] download failed: org.mockito#mockito-all;1.9.0!mockito-all.doc
fehguy commented 10 years ago

Hi, should this be opened in the mokito project?

dragisak commented 10 years ago

It's not a problem with mockito-all. Possibly a problem with sbt.

I opened this issue here because I don't think swagger project should include "mockito-all" together with sources and javadoc. How does swagger depend on documentation or source in mockito-all ?

jamhall commented 10 years ago

When will a snapshot be updated for this fix? thanks!

fehguy commented 10 years ago

Hi, it was released on the 12th--if you follow here:

https://groups.google.com/forum/#!forum/swagger-swaggersocket

or on twitter here:

https://twitter.com/reverbfordevs

You'll see the announcements.

jamhall commented 10 years ago

Ah, excellent! Thanks for the quick reply.

d00mko commented 10 years ago

I'm sorry but it still fails under Play 2.2.1 sbt.ResolveException: download failed: org.mockito#mockito-all;1.9.0!mockito-all.src

I've used JDK 1.7

jamhall commented 10 years ago

I'm on play 2.2.1. Here's my deps:

        ("com.wordnik" % "swagger-play2_2.10" % "1.3.1"),
        ("com.wordnik" % "swagger-play2-utils_2.10" % "1.3.1").excludeAll(ExclusionRule(organization = "org.slf4j"))

Note: I made the exclusion rule because of multiple SLF4J bindings.

dragisak commented 10 years ago

Can you figure where mockito-all comes from ?

It's definitely not in swagger-play2 or swagger-play2-utils 1.3.1. You can see pom file in Maven repo.

To get dependency report, you can try sbt-dependency-graph plugin.

andyoll commented 10 years ago

this is the commit that got rid of it: https://github.com/wordnik/swagger-core/commit/a9a9adef8e92f741b07bc21143f61c142415b568

any deployed builds of this branch before that commit will have the problem.

I added it - so apologies to all who have suffered pain as a result..

On 27 November 2013 17:52, Dragisa Krsmanovic notifications@github.comwrote:

Can you figure where mockito-all comes from ?

It's definitely not in swagger-play2 or swagger-play2-utils 1.3.1. You can see pom file in Maven repo.

To get dependency report, you can try sbt-dependency-graph plugin.

— Reply to this email directly or view it on GitHubhttps://github.com/wordnik/swagger-core/issues/348#issuecomment-29405260 .