Fix #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).

Author: philwo

tools/android/BUILD.tools

@@ -1,5 +1,4 @@
 load("//tools/python:private/defs.bzl", "py_library", "py_binary")
-load("//tools/python:private/version.bzl", "PY_BINARY_VERSION")
 
 package(default_visibility = ["//visibility:public"])
 
@@ -277,21 +276,18 @@ alias(
 py_binary(
     name = "instrumentation_test_check",
     srcs = ["instrumentation_test_check.py"],
-    python_version = PY_BINARY_VERSION,
     deps = ["//third_party/py/abseil"],
 )
 
 py_binary(
     name = "build_incremental_dexmanifest",
     srcs = [":build_incremental_dexmanifest.py"],
-    python_version = PY_BINARY_VERSION,
     deps = [],
 )
 
 py_binary(
     name = "build_split_manifest",
     srcs = ["build_split_manifest.py"],
-    python_version = PY_BINARY_VERSION,
     deps = [
         "//third_party/py/abseil",
     ],
@@ -300,7 +296,6 @@ py_binary(
 py_binary(
     name = "incremental_install",
     srcs = ["incremental_install.py"],
-    python_version = PY_BINARY_VERSION,
     deps = [
         "//third_party/py/concurrent:futures",
         "//third_party/py/abseil",
@@ -310,7 +305,6 @@ py_binary(
 py_binary(
     name = "strip_resources",
     srcs = ["strip_resources.py"],
-    python_version = PY_BINARY_VERSION,
     deps = [
         "//third_party/py/abseil",
     ],
@@ -321,7 +315,6 @@ py_binary(
     srcs = [
         "aar_native_libs_zip_creator.py",
     ],
-    python_version = PY_BINARY_VERSION,
     deps = [
         ":junction_lib",
         "//third_party/py/abseil",
@@ -331,7 +324,6 @@ py_binary(
 py_binary(
     name = "stubify_manifest",
     srcs = ["stubify_manifest.py"],
-    python_version = PY_BINARY_VERSION,
     deps = [
         "//third_party/py/abseil",
     ],
@@ -340,7 +332,6 @@ py_binary(
 py_binary(
     name = "aar_embedded_jars_extractor",
     srcs = ["aar_embedded_jars_extractor.py"],
-    python_version = PY_BINARY_VERSION,
     deps = [
         ":junction_lib",
         "//third_party/py/abseil",
@@ -350,7 +341,6 @@ py_binary(
 py_binary(
     name = "aar_resources_extractor",
     srcs = ["aar_resources_extractor.py"],
-    python_version = PY_BINARY_VERSION,
     deps = [
         ":junction_lib",
         "//third_party/py/abseil",
@@ -360,7 +350,6 @@ py_binary(
 py_binary(
     name = "resource_extractor",
     srcs = ["resource_extractor.py"],
-    python_version = PY_BINARY_VERSION,
 )
 
 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,6 @@ py_test(
         "testenv.py",
     ],
     data = [":archive_testdata"],
-    python_version = PY_BINARY_VERSION,
     srcs_version = "PY2AND3",
     tags = [
         # archive.py requires xzcat, which is not available by default on Mac
@@ -64,7 +62,6 @@ py_test(
 py_binary(
     name = "build_tar",
     srcs = ["build_tar.py"],
-    python_version = PY_BINARY_VERSION,
     srcs_version = "PY2AND3",
     visibility = ["//visibility:public"],
     deps = [
@@ -79,7 +76,6 @@ 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,
     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__"],
 )