Add CcInfo to allowed providers of py_library deps.

[lgulich]

Hi Aspect Team

This PR enables better compatibility with pybind11_bazel and makes the aspect py rules more of a drop-in replacement of the official rules_python. Please let me know if you are open to this contribution, then I can also add tests for it.

More specifically: This change is useful when using a
pybind_extension in the deps of a py_libary as the pybind_extension is just a cc_binary under the hood. This PR enables using it like so:

load("@pybind11_bazel//:build_defs.bzl", "pybind_extension")
load("@aspect_rules_python//python:defs.bzl", aspect_py_library="py_library")


pybind_extension(
    name = "my_pybind_extension",
    srcs = ["my_binding.cpp"],
)

aspect_py_library(
    name = "my_python_library",
    srcs = [":my_python_library.py"],
    deps = [":my_pybind_extension"],
)

The py_libary from the official rules_python also already supports using CcInfo providers in the deps attribute.

Without this change a workaround is to wrap the pybind_extension in a py_library from the official rules_python:

load("@pybind11_bazel//:build_defs.bzl", "pybind_extension")
load("@rules_python//python:defs.bzl", official_py_library="py_library")
load("@aspect_rules_python//python:defs.bzl", aspect_py_library="py_library")


pybind_extension(
    name = "my_pybind_extension",
    srcs = ["my_binding.cpp"],
)

official_py_library(
    name = "my_pybind_extension_wrapper",
    srcs = [],
    deps = [":my_pybind_extension"],
)

aspect_py_library(
    name = "my_python_library",
    srcs = [":my_python_library.py"],
    deps = [":my_pybind_extension_wrapper"],
)

---

Changes are visible to end-users: no

Test plan

TODO

[CLAassistant]

All committers have signed the CLA.

[aspect-workflows]

Test

1 test target passed

Targets

//py/tests/cc-deps:test_smoke [k8-fastbuild]                 1s

Total test execution time was 1s. 30 tests (96.8%) were fully cached saving 1m 14s.

[thesayyn]

This sounds a like a reasonable addition. go for it.

[thesayyn]

Thank you for this :)

[lgulich]

@thesayyn thanks for merging this! When is the next release planned that will include this?