Projet

Général

Profil

Development #25596

numéro de version compatible pep-440

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
07 août 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Ça a été fait dans gadjo mais pas ailleurs.


Fichiers

Révisions associées

Révision 1d758c85 (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

misc: generate a version number that's compatible with PEP 440 (#25596)

Historique

#1

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

#2

Mis à jour par Emmanuel Cazenave il y a plus de 5 ans

Devrait être testé dans combo-wip pour que vérifier que le git rev-list passe : en bossant sur #24646,
je me suis rendu compte que le git describe échouait à tous les coups et qu'on passait en permanence par le fallback return '0'
qui n'aura plus lieu avec ce patch.

(pourquoi ça échoue je ne me l'explique pas et bien sûr ça n'a pas lieu quand eobuilder construit le paquet, juste quand on passe par tox).

#3

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

Devrait être testé dans combo-wip pour que vérifier que le git rev-list passe : en bossant sur #24646, je me suis rendu compte que le git describe échouait à tous les coups et qu'on passait en permanence par le fallback return '0' qui n'aura plus lieu avec ce patch.

J'ai du mal à te suivre, ce code fonctionne pour gadjo, avec eobuilder, qui est l'endroit où on a besoin d'un numéro de version, et fonctionne avec tox (testé localement); si tu as des problèmes dans #24646 sans ce patch, c'est lié à #24646, pas ce patch.

#4

Mis à jour par Emmanuel Cazenave il y a plus de 5 ans

  • Statut changé de Solution proposée à Solution validée

J'essaie de reformuler : dans un job jenkinsfile, lorsque le get_version est invoqué par tox, le git describe échoue systématiquement, pour une raison qui m'échappe (reproduit à la main sur jenkins2: "fatal: aucun nom trouvé, impossible de décrire quoi que ce soit.")

Avec ton patch si le git describe échoue, un git rev-list sera tenté, et comme tu l'invoques dans un check_output, si la commande échoue, boum terminé.

Comme je ne comprends pourquoi le git describe échoue, je me dis qu'il en sera peut-être de même pour le git rev-list, et que donc il serait utile de tester d'abord dans une branche.

Mais comme je suis un gars volontaire j'ai testé à la main sur jenkins2 et le git rev-list passe, donc vas-y.

#5

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 1d758c85fffa748cc3a8e33684c63da8eb26b944
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Aug 4 11:27:32 2018 +0200

    misc: generate a version number that's compatible with PEP 440 (#25596)
#6

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF