Project

General

Profile

Development #30520

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

Added by Benjamin Dauvergne 5 months ago. Updated 5 months ago.

Status:
Nouveau
Priority:
Bas
Assignee:
-
Category:
-
Target version:
-
Start date:
11 Feb 2019
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No
Demande du club utilisateur:
No

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.

History

#1 Updated by Benjamin Dauvergne 5 months ago

  • Priority changed from Normal to Bas

#2 Updated by Frédéric Péters 5 months ago

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 Updated by Frédéric Péters 5 months ago

  • Demande du club utilisateur set to No
  • Project changed from Interne to Publik

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

#4 Updated by Benjamin Dauvergne 5 months ago

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

Also available in: Atom PDF