Bug #13954
authentic vs gadjo vs setuptools
0%
Description
Sur une mise à jour de la dev,
raise PluginError('unable to load entrypoint %s' % entrypoint, e) authentic2.plugins.PluginError: ('unable to load entrypoint authentic2-idp-saml2 = authentic2.idp.saml:Plugin', VersionConflict(gadjo 0.31.2.g11dd7d4 (/usr/lib/python2.7/dist-packages), Requirement.parse('gadjo>=0.6')))
Il y a plein de trucs compliqués derrière, je serais pour contourner pour le moment, en retirant le '>=0.6'.
Fichiers
Demandes liées
Historique
Mis à jour par Thomas Noël il y a plus de 7 ans
- Fichier 0001-setup.py-bypasses-a-pep440-problem-with-gadjo-versio.patch 0001-setup.py-bypasses-a-pep440-problem-with-gadjo-versio.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
C'est l'histoire du +g au lieu de .g, le plus simple ce serait de faire un tag (oui je sais ça passe le truc en recette...) là tout de suite maintenant.
Mis à jour par Frédéric Péters il y a plus de 7 ans
On ne peut bien sûr pas faire un tag à chaque commit gadjo.
Mis à jour par Frédéric Péters il y a plus de 7 ans
Oui ce n'est pas idéal mais c'est très simple à faire : on pourrait vraiment retirer le numéro de version (qui n'est de toute façon plus correct) de la dépendance dans le setup.py d'authentic ?
Mis à jour par Thomas Noël il y a plus de 7 ans
Ouaich ; c'est d'ailleurs ce que proposait mon patch.
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
Ok, mais bon j'ai ça aussi dans les cartons :
commit b2327fd42411dd6da3d6aea4d8154f902d0dd75e Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Fri Jul 1 11:05:29 2016 +0200 setup.py: follow PEP0440 diff --git a/setup.py b/setup.py index 9f67b4d..61510f3 100644 --- a/setup.py +++ b/setup.py @@ -42,7 +42,7 @@ def get_version(): stdout=subprocess.PIPE, stderr=subprocess.PIPE) result = p.communicate()[0] if p.returncode == 0: - return result.split()[0][1:].replace('-', '.') + return result.split()[0][1:].replace('-g', '+g').replace('-', '.') else: return '0.0.0-%s' % len( subprocess.check_output(
Dans mon souvenir ça impacte setuptools < 20 (qui est dans jessie-backports et implémente la PEP440, la version par défaut sur jessie est 5.5.1 qui n'aime pas ça).
Comme pour gunicorn je serai pour avoir un fichier /etc/apt/preferences distribué par puppet qui indique qu'on veut la version jessie-backports).
Mis à jour par Frédéric Péters il y a plus de 7 ans
Il y a quand même des conséquences qui sont bien moins mesurées.
Je viens de regarder et j'ai déjà fait le travail côté scrutiny, http://git.entrouvert.org/scrutiny.git/commit/?id=669c74584f4455e16ee76129917a5a4e8c8d44af ; mais eobuilder, il y a #12443 qui a été marqué fermé mais après le patch a été reverté, c'est quoi la situation ?
Mis à jour par Thomas Noël il y a plus de 7 ans
Frédéric Péters a écrit :
c'est quoi la situation ?
Je suis aussi un peu perdu.
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
Le patch était inutile car servant juste à cacher un bug dans le setuptools de jessie qui n'est plus dans celui de jessie-backports. Juste qu'il fallait amener les chroots eobuilder à charger le setuptools de jessie-backports.
Mis à jour par Frédéric Péters il y a plus de 7 ans
Et donc, on se tente là le changement dans la numérotation en se disant que rien de notre infra ne va casser et que si ça casse il y aura bien quelqu'un pour réparer ?
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
Il y a le problème wheezy/jessie aussi, si on st sur de pouvoir laisser wheezy derrière pour tout nos paquets de dév, ça me va.
Mis à jour par Frédéric Péters il y a plus de 7 ans
Il y a chez imio encore un peu de wheezy pour la prod de Liège; attendons un peu alors. (en retirant le >=0.6, je te laisse juge).
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
J'ai poussé ton patch mais je ne pense pas qu'il faille fermer le ticket; la disparition des dernières wheezy me semble une bonne tâche de départ.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de En cours à Fermé
C'est maintenant ok (#12641).