Projet

Général

Profil

Bug #43797

mise à jour setup.py

Ajouté par Frédéric Péters il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
09 juin 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Parce que les numéros de versions produits sont pas ok en python 3 (ex: "The version specified ("b'4.74.4.g4c8e0984.dirty'") is an invalid version,").

C'est fait en copiant les bouts importants depuis le setup.py de Chrono, avec déplacement de la fonction pour réduire le diff entre les projets.


Fichiers


Demandes liées

Lié à Passerelle - Development #43753: debian : tourner en python 3Fermé08 juin 2020

Actions

Révisions associées

Révision e0359b8e (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

build: update setup.py for python 3 (#43797)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 4 ans

#2

Mis à jour par Frédéric Péters il y a presque 4 ans

#3

Mis à jour par Thomas Noël il y a presque 4 ans

Selon moi il faut garder le « curdir = os.getcwd() ... os.chdir(curdir) » autour du chdir, parce que path qui est un chemin, le chdir ne fonctionnera qu'une seule fois. Bon, je vois que ça n'est pas présent dans chrono et ça m'interroge.

#4

Mis à jour par Emmanuel Cazenave il y a presque 4 ans

Pas compris la réserve, path est un chemin, qui change à chaque itération, et donc nouveau chdir à chaque fois, il me semble.

#5

Mis à jour par Frédéric Péters il y a presque 4 ans

Sur ce bout de code, pourquoi il est ainsi et plus comme dans passerelle avant, à chaque itération on modifiait le chemin, on faisait le call_command, on revenait au chemin qui était là juste avant. Mais si le call_command() échouait il n'était jamais remis comme il était au début.

Dans Chrono j'ai vu ça via #36331 (ticket django 2.2), je commentais :

7c9dd90 misc: correctly reset current directory after translations

Pas vraiment lié à tout le reste, juste un bug sur lequel je suis tombé à un moment où compilemessages échouait.

#6

Mis à jour par Emmanuel Cazenave il y a presque 4 ans

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

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit e0359b8eb2e4a4bced2b30df91bbfddca62b7359
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Jun 9 08:57:28 2020 +0200

    build: update setup.py for python 3 (#43797)
#8

Mis à jour par Frédéric Péters il y a presque 4 ans

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

Formats disponibles : Atom PDF