Update upload.py patch, add docs and example config files for running production server

Patch for Rietveld example running standalone with gae2django

[Ilia Mirkin, 2011-01-12 18:54:05]

Just a few minor English corrections. Otherwise seems fine. I'll leave it up to
Andi as to whether he wants to include these sample files in gae2django,
although I don't see the harm.

http://codereview.appspot.com/3760048/diff/1/README
File README (right):

http://codereview.appspot.com/3760048/diff/1/README#newcode83
README:83: The preferred method to deploy Django applications is to WSGI
supporting
to ? WSGI

Also, I've read all over that the runserver thing is not preferred. However I've
been using it in an internal environment for ~6 months. Seems fine.

http://codereview.appspot.com/3760048/diff/1/README#newcode88
README:88: only in development mode. For running Rietveld in production
environment, you
in _a_ production env

http://codereview.appspot.com/3760048/diff/1/README#newcode89
README:89: need to setup your web-server to serve /static/ alias directly.
to serve _the_ /static/ alias

http://codereview.appspot.com/3760048/diff/1/README#newcode93
README:93: You may want to take a look at example configuration for running
Rietveld with
take a look at _the_ example config

[techtonik, 2011-01-12 19:15:59]

Uploaded new patch set, this time it also includes information about adding
users.

http://codereview.appspot.com/3760048/diff/1/README
File README (right):

http://codereview.appspot.com/3760048/diff/1/README#newcode83
README:83: The preferred method to deploy Django applications is to WSGI
supporting
On 2011/01/12 18:54:05, Ilia Mirkin wrote:
> to ? WSGI

Deploy to server. No? Deploy on WSGI supporting server?

> Also, I've read all over that the runserver thing is not preferred. However
I've
> been using it in an internal environment for ~6 months. Seems fine.

These advices were given from security point of view. I didn't bother myself
with Apache configs if I were in internal environment too.

http://codereview.appspot.com/3760048/diff/1/README#newcode93
README:93: You may want to take a look at example configuration for running
Rietveld with
On 2011/01/12 18:54:05, Ilia Mirkin wrote:
> take a look at _the_ example config

Got any article about articles? I've got a feeling that I need a fix. =)

[imirkin_alum.mit.edu, 2011-01-12 19:29:18]

On Wed, Jan 12, 2011 at 2:16 PM,  <techtonik@gmail.com> wrote:
> Reviewers: Andi Albrecht, Ilia Mirkin,
>
> Message:
> Uploaded new patch set, this time it also includes information about
> adding users.
>
>
> http://codereview.appspot.com/3760048/diff/1/README
> File README (right):
>
> http://codereview.appspot.com/3760048/diff/1/README#newcode83
> README:83: The preferred method to deploy Django applications is to WSGI
> supporting
> On 2011/01/12 18:54:05, Ilia Mirkin wrote:
>>
>> to ? WSGI
>
> Deploy to server. No? Deploy on WSGI supporting server?

I was just pointing out that a word was missing. Perhaps "to use a WSGI ..."

> Got any article about articles? I've got a feeling that I need a fix. =)

I assume your native tongue is Russian. English has concepts that
Russian does not (and probably even more vice-versa). In this case,
there are three situations: use a definite article ("the"), use an
indefinite article ("a"), and use no article. See
http://esl.about.com/od/thebasics/a/a_an_the.htm for some examples.
But it's really rather hard to get it right... most people I know who
learn English as adults never end up getting it right. A lot of the
time it's just "feel" which is hard to quantify into rules.

But it's probably not as bad as counters in Japanese --
http://en.wikipedia.org/wiki/Japanese_counter_word#Full_list_of_counters
:)

-- 
Ilia Mirkin
imirkin@alum.mit.edu

[techtonik, 2011-01-17 15:22:48]

Andi, do you have time for review this week?

[Andi Albrecht, 2011-01-17 19:44:00]

[hehe, this review is on my todo list for today... :-)]

I've already had a quick look at it and I'm fine with it in general.

Ilia, thanks for having a close look at all this grammar/wording/spelling stuff!
Since I'm another non-native speaker I really appreciate such (really helpful)
comments! Feel free to commit it if you don't have anymore comments.

http://codereview.appspot.com/3760048/diff/17001/README
File README (right):

http://codereview.appspot.com/3760048/diff/17001/README#newcode83
README:83: The preferred method to deploy Django applications is to WSGI
supporting
"is to WSGI supporting web server" sounds a bit strange. Wasn't there a
suggestion by Ilia in the first changeset?

http://codereview.appspot.com/3760048/diff/17001/README#newcode94
README:94: with Apache using modwsgi in apache.conf.example
Let's that the user and path (wsgi process group too?) in apache.conf.example
and codereview.wsgi.example should be reviewed before running. Just to go for
sure :)

[techtonik, 2011-01-18 21:09:00]

Done.

http://codereview.appspot.com/3760048/diff/17001/README
File README (right):

http://codereview.appspot.com/3760048/diff/17001/README#newcode83
README:83: The preferred method to deploy Django applications is to WSGI
supporting
On 2011/01/17 19:44:00, Andi Albrecht wrote:
> "is to WSGI supporting web server" sounds a bit strange. Wasn't there a
> suggestion by Ilia in the first changeset?

Done.

http://codereview.appspot.com/3760048/diff/17001/README#newcode94
README:94: with Apache using modwsgi in apache.conf.example
On 2011/01/17 19:44:00, Andi Albrecht wrote:
> Let's that the user and path (wsgi process group too?) in apache.conf.example
> and codereview.wsgi.example should be reviewed before running. Just to go for
> sure :)

Done.

[Andi Albrecht, 2011-01-20 09:25:38]

Committed r157. Thanks, Anatoly!

(I hope that the wording if ok now, otherwise Google Code now has a nice web
interface to change it :)

[techtonik, 2011-01-20 21:50:53]

Wow! That's extremely useful. If they continue to do these things, I'll have
nothing to do soon. =) I have the same idea for Python documentation long ago.

http://code.google.com/p/support/issues/detail?id=3603
http://code.google.com/p/support/issues/detail?id=3830