Merge branch 'main' of https://github.com/bazel-contrib/rules_python into feat.build.data

Author: rickeylev

CHANGELOG.md

@@ -64,7 +64,8 @@ END_UNRELEASED_TEMPLATE
 
 {#v0-0-0-fixed}
 ### Fixed
-* Nothing fixed.
+* (tests) No more coverage warnings are being printed if there are no sources.
+  ([#2762](https://github.com/bazel-contrib/rules_python/issues/2762))
 
 {#v0-0-0-added}
 ### Added

python/private/BUILD.bazel

@@ -29,6 +29,8 @@ package(
 
 licenses(["notice"])
 
+exports_files(["runtime_env_toolchain_interpreter.sh"])
+
 filegroup(
     name = "distribution",
     srcs = glob(["**"]) + [

python/private/py_executable.bzl

@@ -678,6 +678,7 @@ def _create_stage2_bootstrap(
         output = output,
         substitutions = {
             "%build_data_file%": runfiles_root_path(ctx, build_data_file.short_path),
+            "%coverage_instrumented%": str(int(ctx.configuration.coverage_enabled and ctx.coverage_instrumented())),
             "%coverage_tool%": _get_coverage_tool_runfiles_path(ctx, runtime),
             "%import_all%": "True" if read_possibly_native_flag(ctx, "python_import_all_repositories") else "False",
             "%imports%": ":".join(imports.to_list()),

python/private/stage2_bootstrap_template.py

@@ -43,9 +43,14 @@
 # string otherwise.
 VENV_SITE_PACKAGES = "%venv_rel_site_packages%"
 
+# Whether we should generate coverage data.
+# string, 1 or 0
+COVERAGE_INSTRUMENTED = "%coverage_instrumented%" == "1"
+
 # runfiles-root-relative path to a file with binary-specific build information
 BUILD_DATA_FILE = "%build_data_file%"
 
+# ===== Template substitutions end =====
 
 class BazelBinaryInfoModule(types.ModuleType):
     BUILD_DATA_FILE = BUILD_DATA_FILE
@@ -65,8 +70,6 @@ def get_build_data(self):
 
 sys.modules["bazel_binary_info"] = BazelBinaryInfoModule("bazel_binary_info")
 
-# ===== Template substitutions end =====
-
 
 # Return True if running on Windows
 def is_windows():
@@ -332,11 +335,14 @@ def _maybe_collect_coverage(enable):
     # We need for coveragepy to use relative paths.  This can only be configured
     # using an rc file.
     rcfile_name = os.path.join(coverage_dir, ".coveragerc_{}".format(unique_id))
+    disable_warnings = ('disable_warnings = module-not-imported, no-data-collected'
+                        if COVERAGE_INSTRUMENTED else '')
     print_verbose_coverage("coveragerc file:", rcfile_name)
     with open(rcfile_name, "w") as rcfile:
         rcfile.write(
             f"""[run]
 relative_files = True
+{disable_warnings}
 source =
 \t{source}
 """