[NaCl SDK] Add keyboard interrupt handlers for python scripts

[NaCl SDK] Add keyboard interrupt handlers for python scripts

This means that users (and developers) who hit ctrl-C
while running out scripts don't see ugly python backtraces
and get exposed to implementation details.

Also, fix inconsisency in download.py regarding the progess
callback.

BUG=None

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=202335

[binji, 2013-05-24 19:13:04]

lgtm

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/build_t...
File native_client_sdk/src/build_tools/build_projects.py (right):

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/build_t...
native_client_sdk/src/build_tools/build_projects.py:253:
buildbot_common.ErrorExit('build_projects: interrupted')
maybe just use sys.argv[0]?

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/tools/c...
File native_client_sdk/src/tools/create_nmf.py (right):

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/tools/c...
native_client_sdk/src/tools/create_nmf.py:641: if not os.path.isfile(filename):
doesn't isfile imply exists? I guess this helps if the user has a directory
named |filename|...

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/tools/o...
File native_client_sdk/src/tools/oshelpers.py (right):

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/tools/o...
native_client_sdk/src/tools/oshelpers.py:528: funcName = args[0]
func_name

[Sam Clegg, 2013-05-24 21:21:44]

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/build_t...
File native_client_sdk/src/build_tools/build_projects.py (right):

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/build_t...
native_client_sdk/src/build_tools/build_projects.py:253:
buildbot_common.ErrorExit('build_projects: interrupted')
On 2013/05/24 19:13:04, binji wrote:
> maybe just use sys.argv[0]?

Done.

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/tools/c...
File native_client_sdk/src/tools/create_nmf.py (right):

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/tools/c...
native_client_sdk/src/tools/create_nmf.py:641: if not os.path.isfile(filename):
On 2013/05/24 19:13:04, binji wrote:
> doesn't isfile imply exists? I guess this helps if the user has a directory
> named |filename|...


Helps to distinguish between the two different errors I think.

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/tools/o...
File native_client_sdk/src/tools/oshelpers.py (right):

https://codereview.chromium.org/16022005/diff/1/native_client_sdk/src/tools/o...
native_client_sdk/src/tools/oshelpers.py:528: funcName = args[0]
On 2013/05/24 19:13:04, binji wrote:
> func_name

Done.

[commit-bot: I haz the power, 2013-05-25 01:25:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sbc@chromium.org/16022005/6001

[commit-bot: I haz the power, 2013-05-26 02:34:18]

Change committed as 202335