Projet

Général

Profil

Development #41401

mettre à jour le venv à chaque install

Ajouté par Benjamin Dauvergne il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
07 avril 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Je ne suis pas fan de clean_venv comme je l'ai dit en conf, un "-U --force-reinstall" me parait plus simple, ça nous apporte qu'on aura via une installation incrémentale un état plus proche de ce qu'obtient quelqu'un avec une installation "from scratch", i.e. les nouveaux utilisateurs n'auront pas à découvrir les soucis à notre place.


Fichiers

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

#2

Mis à jour par Emmanuel Cazenave il y a environ 4 ans

Comme je l'ai dit aussi, pas du tout fan que ce soit le comportement par défaut, pour moi c'est le boulot des tests unitaires.

Mais si tu veux reprendre ici #41069 pour que le flag clean_venv fasse un -U --force_reinstall plutôt que shooter le venv, ok.

#3

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

  • Statut changé de Solution proposée à Rejeté

Je ne comprends pas le rapport avec les tests unitaires, c'est même pas les mêmes règles qui s'appliquent. Bon de toute façon ça ne marche avec combo à cause de django-ckeditor (je suppose que le pip -U force une récupération du paquet sur pypi :/).

#4

Mis à jour par Emmanuel Cazenave il y a environ 4 ans

Benjamin Dauvergne a écrit :

Je ne comprends pas le rapport avec les tests unitaires.

L'essentiel des dépendances sont quand même déclarées dans les setup.py et tox en tient compte lui aussi !?

#5

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

Emmanuel Cazenave a écrit :

Benjamin Dauvergne a écrit :

Je ne comprends pas le rapport avec les tests unitaires.

L'essentiel des dépendances sont quand même déclarées dans les setup.py et tox en tient compte lui aussi !?

Sauf quand c'est pas le cas (django-ckeditor) par exemple... c'est justement pour mesurer cet écart que ça a un intérêt.

Formats disponibles : Atom PDF