The Makefile runs python at parse time for every target

Chiusa Adatta ai principianti
#1,291 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Una pull request collegata è già stata integrata.

  • #1298 di @isaaclimmy — integrata

Valutazione

Difficoltà
1/5
Tempo stimato
Meno di un'ora
Idoneità per principianti
88/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python
Ambito
build-system

Direzione di ricerca

Leggi Makefile in corrispondenza delle definizioni di XDIST e XDIST_ENV. Modifica la valutazione delle variabili in modo che Python venga eseguito solo per i target che lo usano, mantenendo coerenti entrambe le variabili. Il lavoro è completato quando make help non avvia alcun interprete; verificalo con il test descritto nell’issue, che fa fallire Python su PATH.

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

Descrizione

enhancement good first issue tooling

Where: Makefile (XDIST).

Current gap: XDIST := $(shell python3 -c ...) is evaluated whenever make reads the file, so even make help or make -n lint starts a Python interpreter just to decide whether pytest-xdist is installed.

Suggested approach: Use a lazily evaluated variable (=) so it is only expanded by the targets that use it, and keep XDIST_ENV consistent with it.

Done when: make help starts no interpreter, checked with a test that runs it with a failing python on PATH.

Lingua principale
HTML
Stelle
47
Fork
45
Merge medio
2h 47m
PR unite (30g)
92

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di yakew7/Fair-Code

Tutte le issue di yakew7/Fair-Code

Issue simili

Altre issue su Build System

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.