Coursier: Remove lock to fix occasional CI failures

tindzk · Jul 6, 2019 · 7fa092f · 7fa092f
1 parent 8509df5
commit 7fa092f
Showing 1 changed file with 1 addition and 7 deletions.
diff --git a/src/main/scala/seed/artefact/Coursier.scala b/src/main/scala/seed/artefact/Coursier.scala
@@ -2,7 +2,6 @@ package seed.artefact
 
 import java.io.File
 import java.nio.file.{Path, Paths}
-import java.util.concurrent.locks.ReentrantLock
 
 import coursier._
 import coursier.ivy.IvyRepository
@@ -47,8 +46,6 @@ object Coursier {
   def coursierDependencies(deps: Set[JavaDep]): Seq[coursier.core.Dependency] =
     deps.map(r => Dependency(Module(Organization(r.organisation), ModuleName(r.artefact)), r.version)).toList
 
-  private val lock = new ReentrantLock()
-
   def resolve(all: Set[JavaDep], resolvers: Resolvers, ivyPath: Path, cachePath: Path, silent: Boolean): Resolution =
     if (all.isEmpty) Resolution.empty
     else {
@@ -141,16 +138,13 @@ object Coursier {
                          cachePath: Path,
                          optionalArtefacts: Boolean,
                          silent: Boolean): ResolutionResult = {
-    lock.lock()
     val resolution = resolve(deps, resolvers, ivyPath, cachePath, silent)
     val artefacts = resolution.dependencyArtifacts(
       Some(overrideClassifiers(
         sources = optionalArtefacts,
         javaDoc = optionalArtefacts))).map(_._3).toList
 
-    val result = ResolutionResult(resolution, localArtefacts(artefacts, cachePath, silent))
-    lock.unlock()
-    result
+    ResolutionResult(resolution, localArtefacts(artefacts, cachePath, silent))
   }
 
   def resolveSubset(resolution: Resolution,