file:// no longer considered valid maven coordinates

cbeust / kobalt

A Kotlin-based build system for the JVM.

Apache License 2.0

433 stars 60 forks source link

file:// no longer considered valid maven coordinates #363

Closed ethauvin closed 7 years ago

ethauvin commented 7 years ago

See #352

cbeust commented 7 years ago

How can this happen, --checkVersions already keeps only Maven coordinates:

CheckVersions.kt:

                if (MavenId.isMavenId(dep.id)) {

ethauvin commented 7 years ago

But it didn't, it just splited the coordinate by : and made sure there was between 3 and 5 elements. That's why I fixed isMavenId. You can test it, before and after. You'll see.