Fix bazelbuild#10127: Remove Python 2 dependency from tools/android.

A couple of files still specified PY2 as their python_version. This
fails on modern systems that do not ship Python 2 anymore, or at least
do not install it by default (like Ubuntu 20.04).

PiperOrigin-RevId: 344244485

third_party/ijar/BUILD

@@ -124,20 +124,6 @@ filegroup(
 
 genrule(
     name = "ijar_transitive_zip",
-    srcs = [
-        ":ijar_srcs_zip",
-        ":ijar_deploy_zip",
-        "//src:zlib_zip",
-        "//src/main/cpp/util:cpp_util_with_deps_zip",
-    ],
-    outs = ["ijar_with_deps.zip"],
-    cmd = "$(location //src:merge_zip_files) - $@ $(SRCS)",
-    tools = ["//src:merge_zip_files"],
-    visibility = ["//visibility:public"],
-)
-
-genrule(
-    name = "ijar_transitive_srcs_zip",
     srcs = [
         ":ijar_srcs_zip",
         "//src:zlib_zip",
@@ -149,24 +135,15 @@ genrule(
     visibility = ["//visibility:public"],
 )
 
-genrule(
-    name = "ijar_deploy_zip",
-    srcs = [
-        ":ijar",
-        ":zipper",
-    ],
-    outs = ["ijar_deploy.zip"],
-    cmd = "$(location //src:zip_files) ijar $@ $(SRCS)",
-    tools = ["//src:zip_files"],
-    visibility = ["//visibility:public"],
-)
-
 genrule(
     name = "ijar_srcs_zip",
     srcs = glob(
         ["**"],
         exclude = ["BUILD"],
-    ),
+    ) + [
+        ":ijar",
+        ":zipper",
+    ],
     outs = ["ijar_srcs.zip"],
     cmd = "$(location //src:zip_files) ijar $@ $(SRCS)",
     tools = ["//src:zip_files"],

tools/android/BUILD.tools

@@ -1,5 +1,4 @@
 load("//tools/python:private/defs.bzl", "py_binary", "py_library")
-load("//tools/python:private/version.bzl", "PY_BINARY_VERSION")
 load(":defs.bzl", "run_ijar", "run_singlejar")
 
 package(default_visibility = ["//visibility:public"])
@@ -269,21 +268,21 @@ alias(
 py_binary(
     name = "instrumentation_test_check",
     srcs = ["instrumentation_test_check.py"],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     deps = ["//third_party/py/abseil"],
 )
 
 py_binary(
     name = "build_incremental_dexmanifest",
     srcs = [":build_incremental_dexmanifest.py"],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     deps = [],
 )
 
 py_binary(
     name = "build_split_manifest",
     srcs = ["build_split_manifest.py"],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     deps = [
         "//third_party/py/abseil",
     ],
@@ -292,7 +291,7 @@ py_binary(
 py_binary(
     name = "incremental_install",
     srcs = ["incremental_install.py"],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     deps = [
         "//third_party/py/abseil",
         "//third_party/py/concurrent:futures",
@@ -302,7 +301,7 @@ py_binary(
 py_binary(
     name = "strip_resources",
     srcs = ["strip_resources.py"],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     deps = [
         "//third_party/py/abseil",
     ],
@@ -313,7 +312,7 @@ py_binary(
     srcs = [
         "aar_native_libs_zip_creator.py",
     ],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     deps = [
         ":junction_lib",
         "//third_party/py/abseil",
@@ -323,7 +322,7 @@ py_binary(
 py_binary(
     name = "stubify_manifest",
     srcs = ["stubify_manifest.py"],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     deps = [
         "//third_party/py/abseil",
     ],
@@ -332,7 +331,7 @@ py_binary(
 py_binary(
     name = "aar_embedded_jars_extractor",
     srcs = ["aar_embedded_jars_extractor.py"],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     deps = [
         ":junction_lib",
         "//third_party/py/abseil",
@@ -342,7 +341,7 @@ py_binary(
 py_binary(
     name = "aar_resources_extractor",
     srcs = ["aar_resources_extractor.py"],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     deps = [
         ":junction_lib",
         "//third_party/py/abseil",
@@ -352,7 +351,7 @@ py_binary(
 py_binary(
     name = "resource_extractor",
     srcs = ["resource_extractor.py"],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
 )
 
 py_library(

tools/build_defs/pkg/BUILD

@@ -2,7 +2,6 @@
 load("//tools/build_defs/pkg:pkg.bzl", "pkg_deb", "pkg_tar")
 load("//tools/config:common_settings.bzl", "bool_flag")
 load("//tools/python:private/defs.bzl", "py_binary", "py_library", "py_test")
-load("//tools/python:private/version.bzl", "PY_BINARY_VERSION")
 
 licenses(["notice"])  # Apache 2.0
 
@@ -43,7 +42,7 @@ py_test(
         "testenv.py",
     ],
     data = [":archive_testdata"],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     srcs_version = "PY2AND3",
     tags = [
         # archive.py requires xzcat, which is not available by default on Mac
@@ -64,7 +63,7 @@ py_test(
 py_binary(
     name = "build_tar",
     srcs = ["build_tar.py"],
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     srcs_version = "PY2AND3",
     visibility = ["//visibility:public"],
     deps = [
@@ -79,7 +78,7 @@ py_binary(
     deprecation = "The internal version of make_deb is deprecated. Please " +
                   "use the replacement for pkg_deb from " +
                   "https://github.com/bazelbuild/rules_pkg/blob/master/pkg.",
-    python_version = PY_BINARY_VERSION,
+    python_version = "PY3",
     srcs_version = "PY2AND3",
     visibility = ["//visibility:public"],
     deps = [

tools/python/BUILD

@@ -12,10 +12,7 @@ filegroup(
 
 filegroup(
     name = "embedded_tools",
-    srcs = glob(
-        ["**"],
-        exclude = ["private/version.bzl"],
-    ) + [
+    srcs = glob(["**"]) + [
         "//tools/python/runfiles:embedded_tools",
     ],
     visibility = ["//tools:__pkg__"],

tools/python/BUILD.tools

@@ -54,7 +54,6 @@ exports_files([
     "utils.bzl",
     "private/defs.bzl",
     "private/py_test_alias.bzl",
-    "private/version.bzl",
     # write_file.bzl fortunately doesn't need to be exposed because it's only
     # used in this BUILD file.
 ])
@@ -64,7 +63,6 @@ filegroup(
     srcs = glob(["*.bzl"]) + [
         "private/defs.bzl",
         "private/py_test_alias.bzl",
-        "private/version.bzl",
     ],
     visibility = ["//tools:__pkg__"],
 )