Projet

Général

Profil

Development #41347

rendre le build compatible python3

Ajouté par Serghei Mihai il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Bas
Assigné à:
Version cible:
-
Début:
06 avril 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Mineur. Avec python3 il y a l'erreur:

  File "get_themes.py", line 9
    print 'static/%s/style.css' % dirname

Il suffit de rajouter des paranthèses.


Fichiers

Révisions associées

Révision 7ea10d9f (diff)
Ajouté par Frédéric Péters il y a environ 4 ans

build: switch everything to Python 3 (#41347)

Historique

#1

Mis à jour par Serghei Mihai il y a environ 4 ans

  • Priorité changé de Normal à Bas
#2

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

  • Assigné à mis à Frédéric Péters

"suffirait".

#3

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

#4

Mis à jour par Paul Marillonnet il y a environ 4 ans

  • Pourquoi garder python-gadjo et python3-gadjo dans l'entête Depends du debian/control ?
  • Je pense qu'il y a (i) possibilité de virer certaines entrées de l'entête Conflicts de ce même fichier et (ii) nécessité de porter la/les autre(s) entrée(s) de cet entête.
#5

Mis à jour par Nicolas Roche il y a environ 4 ans

Au cas où, je vois encore 4 occurrences du sheebang '#! /usr/bin/env python' qu'ailleurs ce patch corrige :

$ find . -name '*.py' -exec grep -H python {} \; | grep -v python3
./src/grandlyon-gnm/render-imgs-signalement.py:#! /usr/bin/env python
./templates/variants/villeurbanne-2018/prepare-template.py:#! /usr/bin/env python
./templates/variants/grenoble-metropole-2019/prepare-template.py:#! /usr/bin/env python
./templates/variants/hautes-alpes-2018/prepare-template.py:#! /usr/bin/env python

#6

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

Pourquoi garder python-gadjo et python3-gadjo dans l'entête Depends du debian/control ?

Par rapport à quoi ? Ne rien mettre ? (réponse : la dépendance existe) ou mettre uniquement python3-gadjo ? (réponse : tout n'est pas encore en python 3).

Je pense qu'il y a (i) possibilité de virer certaines entrées de l'entête Conflicts de ce même fichier et (ii) nécessité de porter la/les autre(s) entrée(s) de cet entête.

Pas compris, mais je dirais hors ticket.

Au cas où, je vois encore 4 occurrences du sheebang '#! /usr/bin/env python' qu'ailleurs ce patch corrige :

Hors build, hors ticket.

#7

Mis à jour par Nicolas Roche il y a environ 4 ans

Ok, dpkg-buildpackage fonctionne chez moi en python3.
(Je ne valide pas, parce que je débarque sur le packaging, mais le coeur y est)
Je signal au passage (au cas où) ces 4 warnings :

dpkg-gencontrol: avertissement: relation < is deprecated: use << or <=

(Par curiosité) à quoi sert de créer le répertoire /etc/combo/settings.d ? Il est déployé vide ici.
$ cat debian/publik-base-theme.dirs 
/etc/combo/settings.d

#8

Mis à jour par Paul Marillonnet il y a environ 4 ans

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

Frédéric Péters a écrit :

Pourquoi garder python-gadjo et python3-gadjo dans l'entête Depends du debian/control ?

Par rapport à quoi ? Ne rien mettre ? (réponse : la dépendance existe) ou mettre uniquement python3-gadjo ? (réponse : tout n'est pas encore en python 3).

Oui au temps pour moi j'ai oublié ce détail qui n'en est pas un…

#9

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 7ea10d9f7ce160a50393b1320b44b38e6a7e3c5d
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Apr 6 10:23:24 2020 +0200

    build: switch everything to Python 3 (#41347)
#10

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

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

Formats disponibles : Atom PDF