Projet

Général

Profil

Development #30520

Nos fonction get_version() PEP440 compatible foire si la version actuelle est taggé + dirty

Ajouté par Benjamin Dauvergne il y a 12 jours. Mis à jour il y a 12 jours.

Statut:
Nouveau
Priorité:
Bas
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
11 fév. 2019
Echéance:
% réalisé:

0%

Patch proposed:
Non
Demande du club utilisateur:
Non

Description

Voir ce patch pondu sur eopayment:

commit 70c1a1b5dbefb0a9e51eb0230e0d4f9394602d3f
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Fri Feb 8 12:57:03 2019 +0100

    setup.py: handle case of a tagged dirty workdir

diff --git a/setup.py b/setup.py
index 8b68680..410b982 100755
--- a/setup.py
+++ b/setup.py
@@ -75,7 +75,11 @@ def get_version():
         if p.returncode == 0:
             result = result.decode('ascii').strip()[1:]  # strip spaces/newlines and initial v
             if '-' in result:  # not a tagged version
-                real_number, commit_count, commit_hash = result.split('-', 2)
+                try:
+                    real_number, commit_count, commit_hash = result.split('-', 2)
+                except ValueError:
+                    real_number, commit_hash = result.split('-', 2)
+                    commit_count = 0
                 version = '%s.post%s+%s' % (real_number, commit_count, commit_hash)
             else:
                 version = result

Il faut a priori appliquer le même changement partout.

Historique

#1 Mis à jour par Benjamin Dauvergne il y a 12 jours

  • Priorité changé de Normal à Bas

#2 Mis à jour par Frédéric Péters il y a 12 jours

Ce serait plutôt bien de foirer, mais plus joliment; où il y a des situations où le get_version doit être exécuté sur un git pas net ?

#3 Mis à jour par Frédéric Péters il y a 12 jours

  • Demande du club utilisateur mis à Non
  • Projet changé de Interne à Publik

(je tape ça dans Publik même si ça touche peut-être un peu au-delà)

#4 Mis à jour par Benjamin Dauvergne il y a 12 jours

Tu fais une modif, tu commites pas, tu lances tox, ça foire sur le pip install du projet.

Formats disponibles : Atom PDF