Projet

Général

Profil

Bug #5752

Fonctionner sans rien faire

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Thomas Noël
Version cible:
-
Début:
17 octobre 2014
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:

Description

J'ai voulu tester un truc dans passerelle, après une mise à jour qui m'a tiré le changement sur les settings.py.

 $ ./manage.py runserver
ERR: No configuration file specified, use --config=/path/to/config.py

Je continue à considérer que ça devrait fonctionner.

Surtout qu'après, c'est un nouvel exercice en frustrations,

 $ ./manage.py runserver --config=/tmp/config.py
ERR: No configuration file specified, use --config=/path/to/config.py

 $ ./manage.py --config=/tmp/config.py runserver
[...]
django.core.exceptions.ImproperlyConfigured: The SECRET_KEY setting must not be empty.

 $ ./manage.py --config=/tmp/config.py runserver
CommandError: You must set settings.ALLOWED_HOSTS if DEBUG is False.

 $ ./manage.py --config=/tmp/config.py runserver
[...]
    db = dbs[DEFAULT_DB_ALIAS]
KeyError: 'default'

Fichiers

0001-use-config_example.py-if-existing-to-allow-quickstar.patch (1,16 ko) 0001-use-config_example.py-if-existing-to-allow-quickstar.patch Frédéric Péters, 17 octobre 2014 14:53
0001-manage.py-try-to-use-local-configs-to-allow-quicksta.patch (2,02 ko) 0001-manage.py-try-to-use-local-configs-to-allow-quicksta.patch Thomas Noël, 18 octobre 2014 12:30
0001-simplify-manage.py-and-quick-start-mode-5752.patch (6,33 ko) 0001-simplify-manage.py-and-quick-start-mode-5752.patch Thomas Noël, 20 octobre 2014 21:49
0001-simplify-manage.py-and-quick-start-mode-5752.patch (7,69 ko) 0001-simplify-manage.py-and-quick-start-mode-5752.patch Thomas Noël, 21 octobre 2014 10:01

Historique

#1

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

Patch proposé mais je trouve toujours que c'est appréciable de rester dans le commun des projets django pour faciliter la prise en main.

#2

Mis à jour par Thomas Noël il y a plus de 9 ans

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

Patch proposé mais je trouve toujours que c'est appréciable de rester dans le commun des projets django pour faciliter la prise en main.

L'idée de base était/est "on fait comme sentry" : on propose une commande qui permet de lancer la machinerie, et même un gars qui ne connait pas Django doit savoir lancer le bousin en lisant juste le README. Sentry cependant fait un peu autrement en utilisant carrément des commandes spécifiques (start, upgrade, ...) parce que sentry fait d'autres choses qu'un simple runserver (start lance le gunicorn, etc).

Pour la prise en main par un dev Django, il manquerait l'explication dans le README, que le config.py est chargé à la fin du settings.py.

Bref, je suis ok pour le patch, et pour moi ça suffirait. Ca ferait un peu comme sentry qui essaye "~/.sentry.cnf" par défaut. Eventuellement essayer d'abord un "config.py" avant "config_example.py" pour éviter que les gens tripotent le config_example.py.

#3

Mis à jour par Thomas Noël il y a plus de 9 ans

Ma proposition (qui évite aussi de planter quand on tape juste ./manage.py)

#4

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

Ack.

#5

Mis à jour par Thomas Noël il y a plus de 9 ans

Rediscuté : l'objectif est d'obtenir un manage.py qui se lance même s'il n'y a pas de --config.

et donc, avoir SECRET_KEY=changeme, DATABASES=sqlite3, DEBUG=True, ALLOWED_HOSTS=* etc dans le default_config.py

#6

Mis à jour par Thomas Noël il y a plus de 9 ans

  • Assigné à mis à Thomas Noël
#7

Mis à jour par Thomas Noël il y a plus de 9 ans

Voici un patch qui simplifie le manage.py, permet un démarrage "classique et sans trop rien faire".

Le README ne contient que la doc du mode "quickstart", le config_example.py méritera sans doute un peu d'attention un jour de pluie.

(priorité haute car j'ai besoin que ça soit validé ce mardi matin merci tout le monde)

#9

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

Pas de tiret entre quick et start, un espace, ou rien. Veut-on vraiment utiliser ’ pour les apostrophes ? Si c'est le cas et avec un commentaire "c'est mieux pour la typo", alors il faut faire attention au reste et supprimer l'espace derrière "in the current directory".

#10

Mis à jour par Thomas Noël il y a plus de 9 ans

Avec les corrections apportées dans le README (trop de copiés-collés). Pour le reste, ça passe ?

#11

Mis à jour par Thomas Noël il y a plus de 9 ans

(oups, ne pas tenir compte des deux derniers fichiers du patch ; ils ne seront pas poussés)

#12

Mis à jour par Thomas Noël il y a plus de 9 ans

  • Statut changé de Nouveau à Résolu (à déployer)
  • Priorité changé de Haut à Normal

C'est validé donc poussé.

commit 1b603959fae304e0b8cbc9fd019c1997fd412b89
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Tue Oct 21 10:33:21 2014 +0200

    simplify manage.py and quick-start mode (#5752)
#13

Mis à jour par Thomas Noël il y a plus de 9 ans

  • Patch proposed changé de Oui à Non
#14

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

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF