bazel-contrib / bazel-contrib/rules_python

Support incompatible_strict_action_env

Aperta
#3,655 0 commenti 1 reazione 0 assegnatari Vedi su GitHub
core-rules
Lingua principale
Starlark
Stelle
688
Fork
721
Merge medio
15h 7m
PR unite (30g)
76

Descrizione

As of Bazel 9, `--incompatible_strict_action_env=true` is the default.

Almost everything works with that set, the one exception is Windows in a particular case. That particular case appears to be when the python program is a data dep of another binary, e.g. an sh_binary with a py_binary in data. What happens is the Bazel launcher can't find the python interpreter in runfiles, so falls back to looking for `python.exe` on `PATH`. When strict action env is enabled, the outer PATH isn't inherited, so python.exe can't be found, and thus the launcher fails to execute.

A direct execution of a python program is ok, since the runfiles structure is present enough that the bazel launcher can find the interpreter.

Other than moving away from the Bazel launcher, I don't see a way to solve this. We've wanted to stop using the bazel launcher anyways, so this isn't a big deal, this case just add additional pressure to do so.

Fundamentally, when a binary is running within another binary's runfiles, the logic to find things is a bit more convoluted. We can use RunEnvironmentInfo so that tests auto-inherit PATH, but this only helps tests. binaries-in-binaries or binaries-run-by-actions wouldn't be affected.

There are two workarounds:

* Set `--incompatible_strict_action_env=false` (restore non-strict env behavior, which includes PATH)
* Set `--action_env=PATH` (only inherit PATH for actions)

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia riproducendo il caso Windows con Bazel 9 e l’ambiente rigoroso delle azioni abilitato: uno sh_binary con un py_binary in data. Leggi il comportamento del launcher di Bazel per quanto riguarda i runfiles e la ricerca dell’interprete, quindi confrontalo con l’esecuzione diretta di Python e l’approccio RunEnvironmentInfo. Il lavoro è completato quando il binario annidato riesce a individuare il proprio interprete Python senza richiedere l’ereditarietà di PATH o il workaround documentato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
python
Ambito
build-system
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.