App-wide command line options about flask-script

[rduplain]

This feature has been added to the "decorators" branch. Will set to resolved post 0.2 release.

---

Original Comment By: Dan Jacob

[rduplain]

I'm thinking about something like this:

{{{
#!python

manager = Manager(create_app)
manager.add_option('-c', '--config', dest='config', required=False)
}}}

Then in your create_app function:

{{{
#!python

def create_app(config=None):
app = Flask(name)
if config:
app.config.from_pyfile(config)

return app
}}}

The "config" argument would not be passed to any commands.

I'll consider this feature for the 0.2 release.

---

Original Comment By: Dan Jacob

[rduplain]

Well, the factory function is exactly what I was thinking of when I suggested this. My shell (fish) makes it just a bit harder to set environment variables per-process, and command line options fit better in my mind with configuration than environment variables. (It would also be useful for, say, Flask-CouchDB-based applications, so you could disable automatically syncing in the factory function with a --no-sync option.)

Probably explaining in the docs, "If you pass an app directly to the Manager constructor, you will not be able to use command line options for your configuration" would be sufficient to avoid confusion.

---

Original Comment By: Matthew Frazier

[rduplain]

(Reply via [email protected]):

I did consider the possibility - the problem is that Flask application
may already been initialized when it is passed to the script, so
passing a config option would not work.

However, one possibility is if you pass a factory function to the
Manager constructor, the function could take a default config argument
passed from the command line.

Another option is to use config.from_envvar:

APP_CONFIG=dev.cfg python manage.py shell

def create_app():
app = Flask(name)
app.from_envvar('APP_CONFIG', silent=True)

This second option should at least be mentioned in the docs.

---

Original Comment By: Dan Jacob