Updates- new JGit, etc

rtyley · rtyley · commit a4bf0885dead · 2025-01-18T14:25:00.000Z
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -13,14 +13,8 @@ jobs:
   test:
     runs-on: ubuntu-latest
     steps:
-      - name: Checkout
-        uses: actions/checkout@v3
-      - name: Setup JDK
-        uses: actions/setup-java@v3
-        with:
-          distribution: corretto
-          java-version: 11
-          cache: sbt
+      - uses: actions/checkout@v4
+      - uses: guardian/setup-scala@v1
       - name: Build and Test
         run: sbt -v test
       - name: Test Summary
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -0,0 +1,17 @@
+name: Release
+
+on:
+  workflow_dispatch:
+
+jobs:
+  release:
+    uses: guardian/gha-scala-library-release-workflow/.github/workflows/reusable-release.yml@v1
+    permissions: { contents: write, pull-requests: write }
+    with:
+      GITHUB_APP_ID: 930725
+      SONATYPE_PROFILE_NAME: 'com.madgag'
+      SONATYPE_CREDENTIAL_HOST: 's01.oss.sonatype.org'
+    secrets:
+      SONATYPE_TOKEN: ${{ secrets.AUTOMATED_MAVEN_RELEASE_SONATYPE_TOKEN }}
+      PGP_PRIVATE_KEY: ${{ secrets.AUTOMATED_MAVEN_RELEASE_PGP_SECRET }}
+      GITHUB_APP_PRIVATE_KEY: ${{ secrets.AUTOMATED_MAVEN_RELEASE_GITHUB_APP_PRIVATE_KEY }}
diff --git a/.tool-versions b/.tool-versions
@@ -0,0 +1 @@
+java corretto-11.0.25.9.1
diff --git a/BUILD.md b/BUILD.md
@@ -4,7 +4,7 @@ can run anywhere Java can.
 Here's a rough set of instructions for building the BFG, if you don't want to use the
 pre-built [downloads](http://rtyley.github.io/bfg-repo-cleaner/#download):
 
-* Install Java JDK 8 or above
+* Install Java JDK 11 or above
 * Install [sbt](https://www.scala-sbt.org/1.x/docs/Setup.html)
 * `git clone git@github.com:rtyley/bfg-repo-cleaner.git`
 * `cd bfg-repo-cleaner`
diff --git a/bfg-benchmark/build.sbt b/bfg-benchmark/build.sbt
@@ -1,4 +1,4 @@
-import Dependencies._
+import Dependencies.*
 
 libraryDependencies ++= guava ++ Seq(
   madgagCompress,
diff --git a/bfg-library/build.sbt b/bfg-library/build.sbt
@@ -1,4 +1,4 @@
-import Dependencies._
+import Dependencies.*
 
 libraryDependencies ++= guava ++ Seq(
   parCollections,
@@ -9,6 +9,6 @@ libraryDependencies ++= guava ++ Seq(
   slf4jSimple,
   lineSplitting,
   scalaGitTest % Test,
-  "org.apache.commons" % "commons-text" % "1.9" % Test
+  "org.apache.commons" % "commons-text" % "1.13.0" % Test
 )
 
diff --git a/bfg-library/src/main/scala/com/madgag/git/bfg/cleaner/LfsBlobConverter.scala b/bfg-library/src/main/scala/com/madgag/git/bfg/cleaner/LfsBlobConverter.scala
@@ -30,7 +30,7 @@ import com.madgag.textmatching.{Glob, TextMatcher}
 import org.eclipse.jgit.internal.storage.file.FileRepository
 import org.eclipse.jgit.lib.{ObjectId, ObjectReader}
 
-import java.nio.charset.Charset
+import java.nio.charset.{Charset, StandardCharsets}
 import java.nio.file.{Files, Path}
 import scala.jdk.StreamConverters._
 import scala.util.{Try, Using}
@@ -50,7 +50,7 @@ class LfsBlobConverter(
 
   val gitAttributesLine = s"$lfsGlobExpression filter=lfs diff=lfs merge=lfs -text"
 
-  implicit val UTF_8 = Charset.forName("UTF-8")
+  implicit val UTF_8: Charset = StandardCharsets.UTF_8
 
   val lfsPointerMemo = MemoUtil.concurrentCleanerMemo[ObjectId]()
   
diff --git a/bfg-library/src/main/scala/com/madgag/git/bfg/model/Commit.scala b/bfg-library/src/main/scala/com/madgag/git/bfg/model/Commit.scala
@@ -6,6 +6,7 @@ import org.eclipse.jgit.lib.Constants.OBJ_COMMIT
 import org.eclipse.jgit.lib._
 import org.eclipse.jgit.revwalk.RevCommit
 
+import java.nio.charset.StandardCharsets.UTF_8
 import java.nio.charset.{Charset, IllegalCharsetNameException, UnsupportedCharsetException}
 import scala.jdk.CollectionConverters._
 
@@ -59,10 +60,10 @@ case class CommitArcs(parents: Seq[ObjectId], tree: ObjectId) {
 
 object CommitNode {
   def apply(c: RevCommit): CommitNode = CommitNode(c.getAuthorIdent, c.getCommitterIdent, c.getFullMessage,
-      try c.getEncoding catch {case e @ (_ : IllegalCharsetNameException | _ : UnsupportedCharsetException) => Constants.CHARSET})
+      try c.getEncoding catch {case e @ (_ : IllegalCharsetNameException | _ : UnsupportedCharsetException) => UTF_8})
 }
 
-case class CommitNode(author: PersonIdent, committer: PersonIdent, message: String, encoding: Charset = Constants.CHARSET) {
+case class CommitNode(author: PersonIdent, committer: PersonIdent, message: String, encoding: Charset = UTF_8) {
   lazy val subject = message.linesIterator.to(LazyList).headOption
   lazy val lastParagraphBreak = message.lastIndexOf("\n\n")
   lazy val messageWithoutFooters = if (footers.isEmpty) message else (message take lastParagraphBreak)
diff --git a/bfg-library/src/test/scala/com/madgag/git/bfg/GitUtilSpec.scala b/bfg-library/src/test/scala/com/madgag/git/bfg/GitUtilSpec.scala
@@ -22,11 +22,12 @@ package com.madgag.git.bfg
 
 import com.madgag.git._
 import com.madgag.git.test._
+import org.eclipse.jgit.internal.storage.file.FileRepository
 import org.scalatest.flatspec.AnyFlatSpec
 import org.scalatest.matchers.should.Matchers
 
 class GitUtilSpec extends AnyFlatSpec with Matchers {
-  implicit val repo = unpackRepo("/sample-repos/example.git.zip")
+  implicit val repo: FileRepository = unpackRepo("/sample-repos/example.git.zip")
 
   "reachable blobs" should "match expectations" in {
     implicit val (revWalk, reader) = repo.singleThreadedReaderTuple
diff --git a/bfg-test/src/main/scala/com/madgag/git/bfg/test/unpackedRepo.scala b/bfg-test/src/main/scala/com/madgag/git/bfg/test/unpackedRepo.scala
@@ -2,10 +2,10 @@ package com.madgag.git.bfg.test
 
 import com.madgag.git._
 import com.madgag.git.test._
-import org.eclipse.jgit.internal.storage.file.{GC, ObjectDirectory}
+import org.eclipse.jgit.internal.storage.file.{FileRepository, GC, ObjectDirectory}
 import org.eclipse.jgit.lib.Constants.OBJ_BLOB
 import org.eclipse.jgit.lib.{ObjectId, ObjectReader, Repository}
-import org.eclipse.jgit.revwalk.{RevCommit, RevTree}
+import org.eclipse.jgit.revwalk.{RevCommit, RevTree, RevWalk}
 import org.eclipse.jgit.treewalk.TreeWalk
 import org.scalatest.Inspectors
 import org.scalatest.flatspec.AnyFlatSpec
@@ -16,9 +16,9 @@ import scala.jdk.CollectionConverters._
 
 class unpackedRepo(filePath: String) extends AnyFlatSpec with Matchers {
 
-  implicit val repo = unpackRepo(filePath)
-  implicit val objectDirectory = repo.getObjectDatabase.asInstanceOf[ObjectDirectory]
-  implicit lazy val (revWalk, reader) = repo.singleThreadedReaderTuple
+  implicit val repo: FileRepository = unpackRepo(filePath)
+  implicit val objectDirectory: ObjectDirectory = repo.getObjectDatabase
+  implicit lazy val (revWalk: RevWalk, reader: ObjectReader) = repo.singleThreadedReaderTuple
 
 
   def blobOfSize(sizeInBytes: Int): Matcher[ObjectId] = Matcher { (objectId: ObjectId) =>
@@ -29,8 +29,8 @@ class unpackedRepo(filePath: String) extends AnyFlatSpec with Matchers {
   }
 
   def packedBlobsOfSize(sizeInBytes: Long): Set[ObjectId] = {
-    implicit val reader = repo.newObjectReader()
-    repo.getObjectDatabase.asInstanceOf[ObjectDirectory].packedObjects.filter { objectId =>
+    implicit val reader: ObjectReader = repo.newObjectReader()
+    repo.getObjectDatabase.packedObjects.filter { objectId =>
       val objectLoader = objectId.open
       objectLoader.getType == OBJ_BLOB && objectLoader.getSize == sizeInBytes
     }.toSet
diff --git a/bfg/build.sbt b/bfg/build.sbt
@@ -1,6 +1,6 @@
 import java.io.{File, FileOutputStream}
 
-import Dependencies._
+import Dependencies.*
 import sbt.taskKey
 
 import scala.sys.process.Process
@@ -13,7 +13,7 @@ gitDescription := Try[String](Process("git describe --all --always --dirty --lon
 libraryDependencies += useNewerJava
 
 mainClass := Some("use.newer.java.Version8")
-packageOptions in (Compile, packageBin) +=
+Compile / packageBin / packageOptions +=
   Package.ManifestAttributes( "Main-Class-After-UseNewerJava-Check" -> "com.madgag.git.bfg.cli.Main" )
 
 // note you don't want the jar name to collide with the non-assembly jar, otherwise confusion abounds.
@@ -25,18 +25,18 @@ buildInfoPackage := "com.madgag.git.bfg"
 
 crossPaths := false
 
-publishArtifact in (Compile, packageBin) := false
+Compile / packageBin / publishArtifact := false
 
 // replace the conventional main artifact with an uber-jar
-addArtifact(artifact in (Compile, packageBin), assembly)
+addArtifact(Compile / packageBin / artifact, assembly)
 
 val cliUsageDump = taskKey[File]("Dump the CLI 'usage' output to a file")
 
 cliUsageDump := {
   val usageDumpFile = File.createTempFile("bfg-usage", "dump.txt")
   val scalaRun = new ForkRun(ForkOptions().withOutputStrategy(CustomOutput(new FileOutputStream(usageDumpFile))))
 
-  val mainClassName = (mainClass in (Compile, run)).value getOrElse sys.error("No main class detected.")
+  val mainClassName = (Compile / run / mainClass).value getOrElse sys.error("No main class detected.")
   val classpath = Attributed.data((Runtime / fullClasspath).value)
   val args = Seq.empty
 
@@ -52,7 +52,7 @@ libraryDependencies ++= Seq(
   scalaGitTest % "test"
 )
 
-import Tests._
+import Tests.*
 {
   def isolateTestsWhichRequireTheirOwnJvm(tests: Seq[TestDefinition]) = {
     val (testsRequiringIsolation, testsNotNeedingIsolation) = tests.partition(_.name.contains("RequiresOwnJvm"))
diff --git a/bfg/src/main/scala/com/madgag/git/bfg/cli/CLIConfig.scala b/bfg/src/main/scala/com/madgag/git/bfg/cli/CLIConfig.scala
@@ -45,7 +45,7 @@ object CLIConfig {
   val parser = new OptionParser[CLIConfig]("bfg") {
 
     def fileMatcher(name: String, defaultType: TextMatcherType = Glob) = {
-      implicit val textMatcherRead = Read.reads { TextMatcher(_, defaultType) }
+      implicit val textMatcherRead: Read[TextMatcher] = Read.reads { TextMatcher(_, defaultType) }
 
       opt[TextMatcher](name).valueName(s"<${defaultType.expressionPrefix}>").validate { m =>
         if (m.expression.contains('/')) {
@@ -143,7 +143,7 @@ case class CLIConfig(stripBiggestBlobs: Option[Int] = None,
 
   lazy val gitdir = resolveGitDirFor(repoLocation)
 
-  implicit lazy val repo = FileRepositoryBuilder.create(gitdir.get).asInstanceOf[FileRepository]
+  implicit lazy val repo: FileRepository = FileRepositoryBuilder.create(gitdir.get).asInstanceOf[FileRepository]
 
   lazy val objectProtection = ProtectedObjectCensus(protectBlobsFromRevisions)
 
diff --git a/bfg/src/test/scala/com/madgag/git/bfg/cli/MainSpec.scala b/bfg/src/test/scala/com/madgag/git/bfg/cli/MainSpec.scala
@@ -23,7 +23,7 @@ package com.madgag.git.bfg.cli
 import com.madgag.git._
 import com.madgag.git.bfg.cli.test.unpackedRepo
 import com.madgag.git.bfg.model._
-import org.eclipse.jgit.lib.ObjectId
+import org.eclipse.jgit.lib.{ObjectId, ObjectReader}
 import org.scalatest.flatspec.AnyFlatSpec
 import org.scalatest.matchers.should.Matchers
 import org.scalatest.{Inspectors, OptionValues}
@@ -36,7 +36,7 @@ class MainSpec extends AnyFlatSpec with Matchers with OptionValues with Inspecto
   // concurrent testing against scala.App is not safe https://twitter.com/rtyley/status/340376844916387840
 
   "CLI" should "not change commits unnecessarily" in new unpackedRepo("/sample-repos/exampleWithInitialCleanHistory.git.zip") {
-    implicit val r = reader
+    implicit val r: ObjectReader = reader
 
     ensureInvariantValue(commitHist() take 2) {
       ensureRemovalFrom(commitHist()).ofCommitsThat(haveCommitWhereObjectIds(contain(abbrId("294f")))) {
@@ -90,7 +90,7 @@ class MainSpec extends AnyFlatSpec with Matchers with OptionValues with Inspecto
   }
 
   "strip blobs by id" should "work" in new unpackedRepo("/sample-repos/example.git.zip") {
-    implicit val r = reader
+    implicit val r: ObjectReader = reader
 
     val badBlobs = Set(abbrId("db59"), abbrId("86f9"))
     val blobIdsFile = Files.createTempFile("test-strip-blobs",".ids")
diff --git a/build.sbt b/build.sbt
@@ -1,15 +1,12 @@
-import Dependencies._
-import common._
+import Dependencies.*
 
 ThisBuild / organization := "com.madgag"
 
-ThisBuild / scalaVersion := "2.13.10"
+ThisBuild / scalaVersion := "2.13.16"
 
 ThisBuild / scalacOptions ++= Seq("-deprecation", "-feature", "-language:postfixOps")
 
-ThisBuild / licenses := Seq("GPLv3" -> url("http://www.gnu.org/licenses/gpl-3.0.html"))
-
-ThisBuild / homepage := Some(url("https://github.com/rtyley/bfg-repo-cleaner"))
+ThisBuild / licenses := Seq(License.GPL3_or_later)
 
 ThisBuild / resolvers ++= jgitVersionOverride.map(_ => Resolver.mavenLocal).toSeq
 
@@ -20,30 +17,12 @@ ThisBuild / Test/testOptions += Tests.Argument(
   "-u", s"test-results/scala-${scalaVersion.value}"
 )
 
-lazy val root = Project(id = "bfg-parent", base = file(".")) aggregate (bfg, bfgTest, bfgLibrary)
-
-releaseSignedArtifactsSettings
-
-lazy val bfgTest = bfgProject("bfg-test")
+lazy val root = Project(id = "bfg-parent", base = file(".")) aggregate (bfg, `bfg-test`, `bfg-library`)
 
-lazy val bfgLibrary = bfgProject("bfg-library") dependsOn(bfgTest % Test)
+lazy val `bfg-test` = project
 
-lazy val bfg = bfgProject("bfg") enablePlugins(BuildInfoPlugin) dependsOn(bfgLibrary, bfgTest % Test)
+lazy val `bfg-library` = project.dependsOn(`bfg-test` % Test)
 
-lazy val bfgBenchmark = bfgProject("bfg-benchmark")
+lazy val bfg = project.enablePlugins(BuildInfoPlugin).dependsOn(`bfg-library`, `bfg-test` % Test)
 
-ThisBuild / publishTo := sonatypePublishToBundle.value
-
-ThisBuild / pomExtra := (
-  <scm>
-    <url>git@github.com:rtyley/bfg-repo-cleaner.git</url>
-    <connection>scm:git:git@github.com:rtyley/bfg-repo-cleaner.git</connection>
-  </scm>
-    <developers>
-      <developer>
-        <id>rtyley</id>
-        <name>Roberto Tyley</name>
-        <url>https://github.com/rtyley</url>
-      </developer>
-    </developers>
-)
+lazy val `bfg-benchmark` = project
diff --git a/project/build.properties b/project/build.properties
@@ -1 +1 @@
-sbt.version=1.8.2
+sbt.version=1.10.7
diff --git a/project/common.scala b/project/common.scala
diff --git a/project/dependencies.scala b/project/dependencies.scala
@@ -2,37 +2,37 @@ import sbt._
 
 object Dependencies {
 
-  val scalaGitVersion = "4.3"
+  val scalaGitVersion = "4.8"
 
   val jgitVersionOverride = Option(System.getProperty("jgit.version"))
 
-  val jgitVersion = jgitVersionOverride.getOrElse("4.4.1.201607150455-r")
+  val jgitVersion = jgitVersionOverride.getOrElse("6.10.0.202406032230-r")
 
   val jgit = "org.eclipse.jgit" % "org.eclipse.jgit" % jgitVersion
 
   // the 1.7.2 here matches slf4j-api in jgit's dependencies
   val slf4jSimple = "org.slf4j" % "slf4j-simple" % "1.7.2"
 
-  val scalaCollectionPlus =  "com.madgag" %% "scala-collection-plus" % "0.5"
+  val scalaCollectionPlus =  "com.madgag" %% "scala-collection-plus" % "0.11"
 
-  val parCollections = "org.scala-lang.modules" %% "scala-parallel-collections" % "1.0.0"
+  val parCollections = "org.scala-lang.modules" %% "scala-parallel-collections" % "1.2.0"
 
   val scalaGit = "com.madgag.scala-git" %% "scala-git" % scalaGitVersion exclude("org.eclipse.jgit", "org.eclipse.jgit")
 
   val scalaGitTest = "com.madgag.scala-git" %% "scala-git-test" % scalaGitVersion
 
-  val scalatest = "org.scalatest" %% "scalatest" % "3.2.2"
+  val scalatest = "org.scalatest" %% "scalatest" % "3.2.19"
 
-  val madgagCompress = "com.madgag" % "util-compress" % "1.33"
+  val madgagCompress = "com.madgag" % "util-compress" % "1.35"
 
-  val textmatching = "com.madgag" %% "scala-textmatching" % "2.5"
+  val textmatching = "com.madgag" %% "scala-textmatching" % "2.8"
 
   val scopt = "com.github.scopt" %% "scopt" % "3.7.1"
 
-  val guava = Seq("com.google.guava" % "guava" % "30.1-jre", "com.google.code.findbugs" % "jsr305" % "2.0.3")
+  val guava = Seq("com.google.guava" % "guava" % "33.4.0-jre", "com.google.code.findbugs" % "jsr305" % "3.0.2")
 
-  val useNewerJava =  "com.madgag" % "use-newer-java" % "0.1"
+  val useNewerJava =  "com.madgag" % "use-newer-java" % "1.0.2"
 
-  val lineSplitting = "com.madgag" %% "line-break-preserving-line-splitting" % "0.1.0"
+  val lineSplitting = "com.madgag" %% "line-break-preserving-line-splitting" % "0.1.6"
 
 }
diff --git a/project/plugins.sbt b/project/plugins.sbt
@@ -1,11 +1,9 @@
-scalacOptions += "-deprecation"
+addSbtPlugin("com.github.sbt" % "sbt-release" % "1.4.0")
 
-addSbtPlugin("com.eed3si9n" % "sbt-assembly" % "0.15.0")
+addSbtPlugin("org.xerial.sbt" % "sbt-sonatype" % "3.12.2")
 
-addSbtPlugin("com.github.gseitz" % "sbt-release" % "1.0.13")
+addSbtPlugin("ch.epfl.scala" % "sbt-version-policy" % "3.2.1")
 
-addSbtPlugin("org.xerial.sbt" % "sbt-sonatype" % "3.9.5")
+addSbtPlugin("com.eed3si9n" % "sbt-assembly" % "2.3.0")
 
-addSbtPlugin("com.jsuereth" % "sbt-pgp" % "2.1.1")
-
-addSbtPlugin("com.eed3si9n" % "sbt-buildinfo" % "0.10.0")
+addSbtPlugin("com.eed3si9n" % "sbt-buildinfo" % "0.13.1")

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-import Dependencies._`
	`1`	`+import Dependencies.*`
`2`	`2`
`3`	`3`	`libraryDependencies ++= guava ++ Seq(`
`4`	`4`	`madgagCompress,`