Issue 1609 : Update DevMan: upload.py doesn't work with executable flag on

Issue 1609 : Update DevMan: upload.py doesn't work with executable flag on

[arnold.koroa, 2014-03-04 16:14:04]

https://codereview.appspot.com/69930047/diff/1/.settings/com.google.appengine...
File .settings/com.google.appengine.eclipse.core.prefs (right):

https://codereview.appspot.com/69930047/diff/1/.settings/com.google.appengine...
.settings/com.google.appengine.eclipse.core.prefs:6: gaeLaunchAppInBrowser=true
revert this change as changing global settings can affect other developer's
setup.

https://codereview.appspot.com/69930047/diff/1/src/main/webapp/dev/devman.html
File src/main/webapp/dev/devman.html (right):

https://codereview.appspot.com/69930047/diff/1/src/main/webapp/dev/devman.htm...
src/main/webapp/dev/devman.html:1271: Invoke python interpreter in the command
line. Make sure that your
You can go straight to the command and place the PATH instructions as
troubleshooting tips to make the section more concise. Alternatively, you can
add the PATH instructions to the point where the user is instructed to install
python (I think it's at the prerequisite section).

https://codereview.appspot.com/69930047/diff/1/src/main/webapp/dev/devman.htm...
src/main/webapp/dev/devman.html:1282: </span><br> Linux/Unix users can modify
PATH using export command and
is modifying PATH necessary for linux? IIRC it's already set properly by default
when you install.

https://codereview.appspot.com/69930047/diff/1/src/main/webapp/dev/devman.htm...
src/main/webapp/dev/devman.html:1285: upload.py --rev=default -t "Issue {issue
number} : {branch
no need to repeat the command as it's already shown above?

[arnold.koroa, 2014-03-06 03:22:54]

https://codereview.appspot.com/69930047/diff/40001/src/main/webapp/dev/devman...
File src/main/webapp/dev/devman.html (right):

https://codereview.appspot.com/69930047/diff/40001/src/main/webapp/dev/devman...
src/main/webapp/dev/devman.html:1274: unused constants from Attributes classes"
The instructions about adding python to PATH is still necessary for windows user
as usually installers don't do it automatically. Please add it to the
prerequisite section here /dev/devman.html#settingup-prerequisites

https://codereview.appspot.com/69930047/diff/40001/src/main/webapp/index.html
File src/main/webapp/index.html (right):

https://codereview.appspot.com/69930047/diff/40001/src/main/webapp/index.html...
src/main/webapp/index.html:61: </div>
What is this icon? Is it from your orientation task? If it is please revert this
change.

[arnold.koroa, 2014-03-06 07:22:14]

https://codereview.appspot.com/69930047/diff/80001/src/main/webapp/dev/devman...
File src/main/webapp/dev/devman.html (right):

https://codereview.appspot.com/69930047/diff/80001/src/main/webapp/dev/devman...
src/main/webapp/dev/devman.html:908: <li>Make sure that your command window
recognises <span
Make this as a part of the previous bullet point? They are about the same topic.

Also "command window" -> "command prompt"

https://codereview.appspot.com/69930047/diff/80001/src/main/webapp/dev/devman...
src/main/webapp/dev/devman.html:911: directory in PATH environment variable. For
Windows, see <a
"If it doesn't, you need to add <span class="code">python</span>'s installation
directory in your PATH environment variable."