mitogen-hq/mitogen

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

Open

#811 geöffnet am 14. Feb. 2021

Auf GitHub ansehen
 (5 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)Python (193 Forks)batch import
affects-0.2affects-0.3cigood first issuepython3

Repository-Metriken

Stars
 (2.210 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 22T 19h) (7 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Identifiziere alle Dateien mit hartcodiertem #!/usr/bin/python Hashbang, ersetze durch #!/usr/bin/env python und füge Kommentare für Ausnahmen hinzu, bei denen der hartcodierte Pfad beabsichtigt ist.
Tech Stack
python
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75