wuw

cache/src/main/scala/coursier/Cache.scala

@@ -957,8 +957,6 @@ object Cache {
     }
   }
 
-  private val utf8Bom = "\ufeff"
-
   def fetch(
     cache: File = default,
     cachePolicy: CachePolicy = CachePolicy.UpdateChanging,
@@ -981,7 +979,7 @@ object Cache {
         def notFound(f: File) = Left(s"${f.getCanonicalPath} not found")
 
         def read(f: File) =
-          try Right(new String(FileUtil.readAllBytes(f), "UTF-8").stripPrefix(utf8Bom))
+          try Right(new String(FileUtil.readAllBytes(f), "UTF-8"))
           catch {
             case NonFatal(e) =>
               Left(s"Could not read (file:${f.getCanonicalPath}): ${e.getMessage}")

core/jvm/src/main/scala/coursier/core/compatibility/package.scala

@@ -14,9 +14,11 @@ package object compatibility {
     def letter = c.isLetter
   }
 
+  private val utf8Bom = "\ufeff"
+
   def xmlParse(s: String): Either[String, Xml.Node] = {
     def parse =
-      try Right(scala.xml.XML.loadString(s))
+      try Right(scala.xml.XML.loadString(s.stripPrefix(utf8Bom)))
       catch { case e: Exception => Left(e.toString + Option(e.getMessage).fold("")(" (" + _ + ")")) }
 
     def fromNode(node: scala.xml.Node): Xml.Node =

tests/shared/src/test/scala/coursier/test/CentralTests.scala

@@ -32,6 +32,14 @@ object CentralTests extends TestSuite {
     )
       .process
       .run(fetch)
+      .map { res =>
+
+        assert(res.metadataErrors.isEmpty)
+        assert(res.conflicts.isEmpty)
+        assert(res.isDone)
+
+        res
+      }
       .runF
   }