mitogen-hq/mitogen

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

Open

#811 ouverte le 14 févr. 2021

Voir sur GitHub
 (5 commentaires) (2 réactions) (0 assignés)Python (193 forks)batch import
affects-0.2affects-0.3cigood first issuepython3

Métriques du dépôt

Stars
 (2 210 stars)
Métriques de merge PR
 (Merge moyen 22j 19h) (7 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Identifiez tous les fichiers avec un hashbang #!/usr/bin/python codé en dur, remplacez le par #!/usr/bin/env python et ajoutez des commentaires pour les exceptions où le chemin codé est intentionnel.
Stack technique
python
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75