Cleanup appengine_config.py

Cleanup appengine_config.py

[M-A, 2012-02-16 15:59:06]

rubberstamp lgtm, assuming it works.

[techtonik, 2012-02-16 16:20:57]

Works fine for my local install. Forgot to add Guido - maybe he has objections
as he is the author. If no objections, I'll commit in a few days.

[GvR, 2012-02-16 17:39:07]

LGTM (though I have no idea what the bug you quote in the comment is about).

[Andi Albrecht, 2012-02-17 04:26:20]

On 2012/02/16 17:39:07, GvR wrote:
> LGTM (though I have no idea what the bug you quote in the comment is about).

I don't see a bug here too. Settings DJANGO_SETTINGS_MODULE is common practice
when setting up a Django environment manually.

[techtonik, 2012-02-17 04:53:47]

The stacktrace described in the bug appears in my 1.6.0 SDK when you first load
http://localhost:8080/_ah/stats and then http://localhost:8080 without setting

os.environ['DJANGO_SETTINGS_MODULE'] = 'settings'


See http://paste.kde.org/423776/

I've recheck with 1.6.2. It is still actual and magic. On page refresh traceback
changes to http://paste.kde.org/423782/

[techtonik, 2012-02-17 06:07:25]

"Django 1.1 requires that the DJANGO_SETTINGS_MODULE environment variable be
set.." http://code.google.com/appengine/docs/python/tools/libraries.html#Django
Updated patch with a correct comment. Will commit right now.

What a timewaste. :sad:

I only wonder if the requirement for all "main" scripts to import
webapp.template before django is still actual?

[techtonik, 2012-02-17 06:15:47]

Committed as a7a7f3bb87.

[gvrpython, 2012-02-18 02:56:37]

On Thu, Feb 16, 2012 at 10:07 PM,  <techtonik@gmail.com> wrote:
> I only wonder if the requirement for all "main" scripts to import
> webapp.template before django is still actual?

Switch to Python 2.7 and you won't need this any more...

-- 
--Guido van Rossum (python.org/~guido)