diff --git a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/maven/DependencyManager.kt b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/maven/DependencyManager.kt index d609c905..03641845 100644 --- a/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/maven/DependencyManager.kt +++ b/modules/kobalt-plugin-api/src/main/kotlin/com/beust/kobalt/maven/DependencyManager.kt @@ -60,12 +60,7 @@ class DependencyManager @Inject constructor(val executors: KobaltExecutors, /** * Create an IClasspathDependency from a Maven id. */ - override fun createMaven(id: String, optional: Boolean) : IClasspathDependency= - if (KobaltMavenResolver.isRangeVersion(id)) { - Kobalt.INJECTOR.getInstance(DependencyManager::class.java).create(id, optional) - } else { - resolver.create(id, optional) - } + override fun createMaven(id: String, optional: Boolean) : IClasspathDependency = resolver.create(id, optional) /** * Create an IClasspathDependency from a path. diff --git a/src/test/kotlin/com/beust/kobalt/maven/DependencyManagerTest.kt b/src/test/kotlin/com/beust/kobalt/maven/DependencyManagerTest.kt index 5755241b..00b263ee 100644 --- a/src/test/kotlin/com/beust/kobalt/maven/DependencyManagerTest.kt +++ b/src/test/kotlin/com/beust/kobalt/maven/DependencyManagerTest.kt @@ -34,6 +34,12 @@ class DependencyManagerTest @Inject constructor(val dependencyManager: Dependenc } } + @Test + fun createId() { + // Caused a StackOverflowException in 0.923 + val id = dependencyManager.createMaven("com.beust.kobalt:kobalt-line-count:jar:(0,]") + } + @Test(description = "Make sure that COMPILE scope dependencies get resolved properly") fun testScopeDependenciesShouldBeDownloaded() { val testDeps = listOf(dependencyManager.create("org.testng:testng:6.10"))