bazel-contrib / bazel-contrib/rules_python
Gazelle python_root directive only adds imports to subfolders
- Lenguaje dominante
- Starlark
- Estrellas
- 688
- Forks
- 721
- Merge medio
- 15 h 7 min
- PR fusionados (30 d)
- 76
Descripción
# 🐞 bug report
### Affected Rule
Gazelle plugin target generation
### Is this a regression?
No
### Description
python targets that do not share root with the workspace needs `imports = [...]` statements added for imports to work correctly. The gazelle plugin supports this through adding `# gazelle:python_root` directives in `BUILD` files that are at the root of python trees. This works great for source files in subfolders, they have attributes like `imports = ["../.."],` added. However the sources files in the root folder itself does not get any imports attribute, while they need to have `imports = ["."]` to work correctly.
## 🔬 Minimal Reproduction
## 🔥 Exception or Error
Target py_binary file fails on run due to missing import.
## 🌍 Your Environment
**Operating System:**
Ubuntu 24.10
**Output of `bazel version`:**
Bazelisk version: v1.25.0
Build label: 8.0.1
Build target: @@//src/main/java/com/google/devtools/build/lib/bazel:BazelServer
Build time: Fri Jan 17 19:16:16 2025 (1737141376)
Build timestamp: 1737141376
Build timestamp as int: 1737141376
**Rules_python version:**
bazel_dep(name = "rules_python", version = "1.1.0")
bazel_dep(name = "rules_python_gazelle_plugin", version = "1.1.0")
**Anything else relevant?**
Guía de contribución
Línea de trabajo
Start by tracing Gazelle plugin target generation for BUILD files containing a `# gazelle:python_root` directive, focusing on root-level Python sources and generated `py_binary` targets. Compare root-folder targets with subfolder targets and verify that completed generation adds `imports = ["."]` where needed, then confirm the target runs without the missing-import failure.
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
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100