From a36233950d887a4932537dc7831544d2608d1d9b Mon Sep 17 00:00:00 2001 From: Richard Levasseur Date: Mon, 2 Dec 2024 16:00:04 -0800 Subject: [PATCH] fix: add missing api distribution target (#2464) The distribution file groups are mostly used by integration tests, not releases, so these filegroups missing doesn't usually cause a problem. This was found when importing rules_python into Google, where filegroups of the rules sources are fed into various tests. --- python/private/BUILD.bazel | 1 + python/private/api/BUILD.bazel | 5 +++++ 2 files changed, 6 insertions(+) diff --git a/python/private/BUILD.bazel b/python/private/BUILD.bazel index 9772089e97..76e3a78778 100644 --- a/python/private/BUILD.bazel +++ b/python/private/BUILD.bazel @@ -30,6 +30,7 @@ licenses(["notice"]) filegroup( name = "distribution", srcs = glob(["**"]) + [ + "//python/private/api:distribution", "//python/private/proto:distribution", "//python/private/pypi:distribution", "//python/private/whl_filegroup:distribution", diff --git a/python/private/api/BUILD.bazel b/python/private/api/BUILD.bazel index 9e97dc2b59..0826b85d9b 100644 --- a/python/private/api/BUILD.bazel +++ b/python/private/api/BUILD.bazel @@ -19,6 +19,11 @@ package( default_visibility = ["//:__subpackages__"], ) +filegroup( + name = "distribution", + srcs = glob(["**"]), +) + py_common_api( name = "py_common_api", # NOTE: Not actually public. Implicit dependency of public rules.