bazel-contrib / bazel-contrib/rules_python
Gazelle python_root directive only adds imports to subfolders
- Vorherrschende Sprache
- Starlark
- Sterne
- 688
- Forks
- 721
- Ø Merge
- 15 Std. 7 Min.
- Gemergte PRs (30 T.)
- 76
Beschreibung
# 🐞 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?**
Beitragsleitfaden
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- build-system
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100