Move run_dev_appserver.py to components/tools and make it reusable.

Move run_dev_appserver.py to components/tools and make it reusable.

Also add a corresponding wrapper script swarming/tools/.

R=maruel@chromium.org
BUG=

Committed: https://code.google.com/p/swarming/source/detail?repo=default&r=b2014ba

[vadimsh, 2014-04-25 04:21:12]

ptal

switch.py is probably next.

https://codereview.appspot.com/91790044/diff/1/services/components/tools/run_...
File services/components/tools/run_dev_appserver.py (right):

https://codereview.appspot.com/91790044/diff/1/services/components/tools/run_...
services/components/tools/run_dev_appserver.py:8: This script is mostly useful
for apps that are composed of multiple modules.
It's almost same code as was in isolate/tools/run_dev_appserver.py.

Changes are: doc string, usage string and |args| and |app_dir| in main.

[M-A, 2014-04-25 12:56:54]

lgtm with a few nits

https://codereview.appspot.com/91790044/diff/1/services/components/tools/run_...
File services/components/tools/run_dev_appserver.py (right):

https://codereview.appspot.com/91790044/diff/1/services/components/tools/run_...
services/components/tools/run_dev_appserver.py:33: help='Path to AppEngine SDK.
Will try to find by itself.')
optional:
Add option --verbose
that would pass
--log_level debug

https://codereview.appspot.com/91790044/diff/1/services/components/tools/run_...
services/components/tools/run_dev_appserver.py:59: ] +
gae_sdk_utils.find_module_yamls(app_dir) + args
Add --require_indexes

That's one thing that annoys me with the unit tests, that it doesn't double
check that the required indexes exist.

https://codereview.appspot.com/91790044/diff/1/services/isolate/tools/run_dev...
File services/isolate/tools/run_dev_appserver.py (right):

https://codereview.appspot.com/91790044/diff/1/services/isolate/tools/run_dev...
services/isolate/tools/run_dev_appserver.py:13: sys.path.insert(0,
os.path.join(APP_DIR, '..', 'components', 'third_party'))
This one is not needed BTW.

https://codereview.appspot.com/91790044/diff/1/services/swarming/tools/run_de...
File services/swarming/tools/run_dev_appserver.py (right):

https://codereview.appspot.com/91790044/diff/1/services/swarming/tools/run_de...
services/swarming/tools/run_dev_appserver.py:13: sys.path.insert(0,
os.path.join(APP_DIR, '..', 'components', 'third_party'))
same, and in a few other scripts.

[vadimsh, 2014-04-25 21:02:18]

https://codereview.appspot.com/91790044/diff/1/services/components/tools/run_...
File services/components/tools/run_dev_appserver.py (right):

https://codereview.appspot.com/91790044/diff/1/services/components/tools/run_...
services/components/tools/run_dev_appserver.py:33: help='Path to AppEngine SDK.
Will try to find by itself.')
On 2014/04/25 12:56:54, M-A wrote:
> optional:
> Add option --verbose
> that would pass
> --log_level debug

Done.

https://codereview.appspot.com/91790044/diff/1/services/components/tools/run_...
services/components/tools/run_dev_appserver.py:59: ] +
gae_sdk_utils.find_module_yamls(app_dir) + args
On 2014/04/25 12:56:54, M-A wrote:
> Add --require_indexes
> 
> That's one thing that annoys me with the unit tests, that it doesn't double
> check that the required indexes exist.

Done.

https://codereview.appspot.com/91790044/diff/1/services/isolate/tools/run_dev...
File services/isolate/tools/run_dev_appserver.py (right):

https://codereview.appspot.com/91790044/diff/1/services/isolate/tools/run_dev...
services/isolate/tools/run_dev_appserver.py:13: sys.path.insert(0,
os.path.join(APP_DIR, '..', 'components', 'third_party'))
On 2014/04/25 12:56:54, M-A wrote:
> This one is not needed BTW.

Strictly speaking yes, but you only know this because you've looked at the
implementation. I want 'components' to be usable with that two sys.path
modifications and only them. Want components? Add two sys.path lines. That's why
I'm keeping it here even though it's not needed.

It's easier to copy-paste this two lines as a hard requirements for using code
from 'components' rather than think what particular module really needs in
sys.path.

Do you have a strong opinion about this?

[M-A, 2014-04-25 21:20:14]

https://codereview.appspot.com/91790044/diff/1/services/isolate/tools/run_dev...
File services/isolate/tools/run_dev_appserver.py (right):

https://codereview.appspot.com/91790044/diff/1/services/isolate/tools/run_dev...
services/isolate/tools/run_dev_appserver.py:13: sys.path.insert(0,
os.path.join(APP_DIR, '..', 'components', 'third_party'))
On 2014/04/25 21:02:19, vadimsh wrote:
> Do you have a strong opinion about this?

No.

[vadimsh, 2014-04-25 21:33:45]

Committed patchset #2 manually as rb2014ba.