Projet

Général

Profil

Bug #24081

supprimer "from global_settings import *"

Ajouté par Anonyme il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
25 mai 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

  • Pour assurer en amont la double compatibilité Django 1.8 et 1.11 dans une configuration Hobo (qui cherche l'existence de TEMPLATE_DEBUG, et qui même si TEMPLATES est défini, il sera ignoré)
  • L'option préférable serait d'importer seulement les variables de global_settings dont on a besoin dans les settings d'Authentic.
    Suite à la discussion (x2) sur Hobo (#24071)

Fichiers


Demandes liées

Lié à Authentic 2 - Development #21489: Fonctionner avec Django 1.11 (et 1.8)Fermé06 avril 2018

Actions
Lié à Hobo - Development #24071: Support de TEMPLATES dès qu'il sont définis par ailleursRejeté24 mai 2018

Actions

Révisions associées

Révision 96d6282d (diff)
Ajouté par Emmanuel Cazenave il y a plus de 5 ans

dont' import everything from global_settings (#24081)

Historique

#1

Mis à jour par Anonyme il y a presque 6 ans

#2

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

  • Lié à Development #24071: Support de TEMPLATES dès qu'il sont définis par ailleurs ajouté
#3

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

Testé sur mon publik local.

J'ai fait le tour de debian-jessie/debin_config.py et debian-jessie/multitenant/debian_config.py pour identifier ce qui pourrait clocher,
d'où le hack sur CACHES.

Aussi fait le tour des serveurs de prod et test pour voir ce qu'il y a dans les /etc/authentic2-multitenant. Pour les autres déploiements je ne sais pas.

Sans doute à merger et pousser en tout début de fenêtre pour avoir le max de temps pour ajuster.

#4

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

Ça

STATICFILES_FINDERS = global_settings.STATICFILES_FINDERS + ('gadjo.finders.XStaticFinder',)

Pour que ça passe plusieurs versions de Django notamment les changements tuple() -> list() (je ne sais plus quand ça arrive, ou si c'est déjà complètement arrivé en 1.8), il faudrait pas convertir en concat_seq(s1, s2) avec

def concat_seq(l1, l1):
    return l1 + l1.__class__(l2)

Sinon ok.

#5

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

Ack pour

X = list(X) + [...]
.

#6

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

ok, je pousserai celui là.

#7

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

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

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

Oh l'ot il valide ses propres patchs...

#9

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

Je croyais que c'était le sens de ton "ack pour ..." !

#10

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

Je te taquine.

#11

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 96d6282d56525d0bc444a9cec98c65a7ef493311
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Wed Jun 6 16:45:50 2018 +0200

    dont' import everything from global_settings (#24081)

#12

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

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

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Solution déployée à Fermé

Formats disponibles : Atom PDF