mitogen-hq/mitogen

Modules and script hard code /usr/bin/python in hashbang

Open

#811 aperta il 14 feb 2021

Vedi su GitHub
 (5 commenti) (2 reazioni) (0 assegnatari)Python (193 fork)batch import
affects-0.2affects-0.3cigood first issuepython3

Metriche repository

Star
 (2210 star)
Metriche merge PR
 (Merge medio 22g 19h) (7 PR mergiate in 30 g)

Descrizione

A number of modules, and scripts in the test suite start with

#!/usr/bin/python

In general it is better to use

#!/usr/bin/env python

because that allows execution inside a virtualenv, or when python is installed elsewhere, e.g. /usr/loca/bin/python.

There may be cases in which we really do want to hard code the path, e.g.

  • testing detection/selection of ansible_python_interpreter (correct name?)
  • mechanics of Ansible module transfer, or recognition .

If so those should be explictly noted in a comment.

Guida contributor

Direzione di ricerca
Identifica tutti i file con hashbang #!/usr/bin/python hardcoded, sostituisci con #!/usr/bin/env python e aggiungi commenti per le eccezioni in cui il percorso hardcoded è intenzionale.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75