Allow relative imports in the `__main__` module
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 77.2k
- Forks
- 35.9k
- Métriques de merge des PR
- Métriques de PR en attente
Description
Feature or enhancement
Proposal:
Expanding on https://github.com/python/cpython/issues/109853#issuecomment-2076845977
@zooba, were you envisioning to do this for all scenarios where a __main__ module is exists? I think that makes sense for consistency, but we may want to consider all scenarios to make sure the UX makes sense.
AFAICT, these should be all of them:
(this refers to relative imports in the same directory, it does not cover relative imports to the parent directory)
- Running a package module via
python -m <module>- Relative imports already work here, no changes needed.
- Running a source or extension module via
python -m <module>- Relative imports don't work here.
- Allowing relative imports here feels a bit weird, especially if we were to enable
-Pby default. For it to run, the module would have to be insys.path, meaning it probably installed rather than being provided by the user, so there isn't much necessity for relative imports.
- Running a package module directly (
python <module.zip>)- Relative imports already work here, no changes needed.
- Running a file directly (
python <script.py>)- Relative imports don't work here.
- Running a command string via
python -c <command>- Relative imports don't work here.
- There's no canonical base directory to use as a reference, should we use the current directory?
- Running a command string via the
stdin- Relative imports don't work here.
- There's no canonical base directory to use as a reference, should we use the current directory?
- Running the repl
- Relative imports don't work here.
- There's no canonical base directory to use as a reference, should we use the current directory?
I think 4), 5), 6), and 7) are the only use-cases that call for relative imports to work. 2) feels a bit weird, but it may be worth supporting for consistency.
I played a bit with the code, and the implementing this doesn't seem much difficult. I was able to write a working PoC targeting only 4) with much difficulty, so I don't think that's a worry.
Considering this, do we think it makes sense to make this change?
If so, should it cover all scenarios, or do we want to keep some of them as-is?
For further reflection, how much of an actual improvement would this actually be over the current behavior of adding the current directory to sys.path?
I can't help but feel a bit like we are trading one weird behavior for another.
Has this already been discussed elsewhere?
No response given
Links to previous discussion of this feature:
No response
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par lire la discussion liée dans l’issue 109853 et comparez les sept scénarios d’exécution de main répertoriés ici, en particulier les fichiers directs, -c, stdin et la REPL. Déterminez quels cas devraient prendre en charge les importations relatives, quel répertoire devrait servir de référence lorsqu’il n’existe aucune base canonique, et si le comportement s’améliore en ajoutant le répertoire courant à sys.path. Le travail est considéré comme terminé lorsque le périmètre pris en charge et l’expérience utilisateur qui en résulte sont convenus avant l’implémentation.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- backend
- Type d'issue
- Fonctionnalité
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 25/100