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 environ 5 ans. Mis à jour il y a 3 mois.

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

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non
Club:
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.


Demandes liées

Lié à Publik - Development #81731: Ajustement sur le calcul des versions Fermé28 septembre 2023

Actions

Historique

#1

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

  • Priorité changé de Normal à Bas
#2

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

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 environ 5 ans

  • Projet changé de Interne à Publik
  • Club mis à Non

(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 environ 5 ans

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

#5

Mis à jour par Frédéric Péters il y a 3 mois

  • Statut changé de Nouveau à Fermé
  • Planning mis à Non

Dupliqué et résolu en #81731.

#6

Mis à jour par Frédéric Péters il y a 3 mois

Formats disponibles : Atom PDF