Add support for .tbz file extension in repository_ctx.download_and_ex…

…tract

The `download_and_extract` function supported `.tar.bz2` files, but did not recognize `.tbz` as a valid file extension for those. This commit simply adds support for this, in the code, in the test and in the doc.

Fixes #17824

Closes #17825.

PiperOrigin-RevId: 520049316
Change-Id: Id4a1716304734f58a06cf5dc8d481642f631f5e1

src/main/java/com/google/devtools/build/lib/bazel/repository/DecompressorValue.java

@@ -102,15 +102,15 @@ static Decompressor getDecompressor(Path archivePath) throws RepositoryFunctionE
       return TarXzFunction.INSTANCE;
     } else if (baseName.endsWith(".tar.zst") || baseName.endsWith(".tzst")) {
       return TarZstFunction.INSTANCE;
-    } else if (baseName.endsWith(".tar.bz2")) {
+    } else if (baseName.endsWith(".tar.bz2") || baseName.endsWith(".tbz")) {
       return TarBz2Function.INSTANCE;
     } else if (baseName.endsWith(".ar") || baseName.endsWith(".deb")) {
       return ArFunction.INSTANCE;
     } else {
       throw new RepositoryFunctionException(
           Starlark.errorf(
               "Expected a file with a .zip, .jar, .war, .aar, .tar, .tar.gz, .tgz, .tar.xz, .txz,"
-                  + " .tar.zst, .tzst, .tar.bz2, .ar or .deb suffix (got %s)",
+                  + " .tar.zst, .tzst, .tar.bz2, .tbz, .ar or .deb suffix (got %s)",
               archivePath),
           Transience.PERSISTENT);
     }

src/main/java/com/google/devtools/build/lib/bazel/repository/starlark/StarlarkBaseExternalContext.java

@@ -564,13 +564,11 @@ public StructImpl download(
             defaultValue = "''",
             named = true,
             doc =
-                "the archive type of the downloaded file."
-                    + " By default, the archive type is determined from the file extension of"
-                    + " the URL."
-                    + " If the file has no extension, you can explicitly specify either \"zip\","
-                    + " \"jar\", \"war\", \"aar\", \"tar\", \"tar.gz\", \"tgz\", \"tar.xz\","
-                    + " \"txz\", \".tar.zst\", \".tzst\", \"tar.bz2\", \".ar\", or \".deb\""
-                    + " here."),
+                "the archive type of the downloaded file. By default, the archive type is"
+                    + " determined from the file extension of the URL. If the file has no"
+                    + " extension, you can explicitly specify either \"zip\", \"jar\", \"war\","
+                    + " \"aar\", \"tar\", \"tar.gz\", \"tgz\", \"tar.xz\", \"txz\", \".tar.zst\","
+                    + " \".tzst\", \"tar.bz2\", \".tbz\", \".ar\", or \".deb\" here."),
         @Param(
             name = "stripPrefix",
             defaultValue = "''",

src/test/java/com/google/devtools/build/lib/bazel/repository/DecompressorValueTest.java

@@ -57,6 +57,8 @@ public void testKnownFileExtensionsDoNotThrow() throws Exception {
     unused = DecompressorValue.getDecompressor(path);
     path = fs.getPath("/foo/.external-repositories/some-repo/bar.baz.tar.bz2");
     unused = DecompressorValue.getDecompressor(path);
+    path = fs.getPath("/foo/.external-repositories/some-repo/bar.baz.tbz");
+    unused = DecompressorValue.getDecompressor(path);
     path = fs.getPath("/foo/.external-repositories/some-repo/bar.baz.ar");
     unused = DecompressorValue.getDecompressor(path);
     path = fs.getPath("/foo/.external-repositories/some-repo/bar.baz.deb");