bazel-contrib / bazel-contrib/rules_python
Emit RunfilesGroupInfo in py_binary targets
- Lenguaje dominante
- Starlark
- Estrellas
- 688
- Forks
- 721
- Merge medio
- 15 h 7 min
- PR fusionados (30 d)
- 76
Descripción
# 🚀 feature request
### Relevant Rules
I want `py_binary` to return an additional provider, [`RunfilesGroupInfo`](https://github.com/hermeticbuild/rules_runfiles_group).
`py_library` could contribute by also returning this provider, but that's more of an implementation detail.
I'd be open to work on this if you agree that this is a good idea.
### Description
I'd like to make packaging of `py_binary` targets in container images more efficient. To that end, I have designed a set of providers and described their semantics in [a new Bazel module](https://github.com/hermeticbuild/rules_runfiles_group).
### Describe the solution you'd like
`py_library` targets should forward `RunfilesGroupInfo` and `RunfilesGroupMetadataInfo` from their deps and add their own runfiles (`srcs`, `data`) to a group.
`py_binary` targets should aggregate runfiles groups from their deps and emit `RunfilesGroupInfo` and `RunfilesGroupMetadataInfo` that contain at least:
* A group for the interpreter + standard library
* One group per third party dependency
Additionally, it could also make sense to produce groups for each `py_library` target in the transitive dependency graph.
### Describe alternatives you've considered
Doing nothing would mean we continue creating (very large) container image layers that contain the full runfiles tree of a `py_binary` target.
Writing custom packaging rules just for rules_python is unsustainable in my opinion.
Guía de contribución
Línea de trabajo
Start by reading the py_library and py_binary rule implementations and the provider semantics in the linked rules_runfiles_group Bazel module. Define how dependencies are forwarded and aggregated, then verify that py_binary emits groups for the interpreter and standard library plus third-party dependencies, with tests covering the requested providers and metadata.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- build-system
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Activo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 48/100