Projet

Général

Profil

Bug #13954

authentic vs gadjo vs setuptools

Ajouté par Frédéric Péters il y a plus de 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
14 novembre 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Club:

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

Lié à Gadjo - Bug #12641: PEP 440Fermé19 juillet 2016

Actions

Historique

#2

Mis à jour par Thomas Noël il y a plus de 7 ans

#3

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.

#4

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.

#5

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 ?

#6

Mis à jour par Thomas Noël il y a plus de 7 ans

Ouaich ; c'est d'ailleurs ce que proposait mon patch.

#7

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).

#8

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 ?

#9

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.

#10

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.

#11

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 ?

#12

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.

#13

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).

#14

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.

#15

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).

#16

Mis à jour par Frédéric Péters il y a plus de 5 ans

Formats disponibles : Atom PDF