Fixes JobMixin method names
Fixes error handling when job-check fails

gui/job_monitor.py:
 - update_jobs:
   Fixes call to debug.critical
 - timerEvent:
   Set updating_now in try/finally block so that it always gets called

Ref #27
Mark controller changed when deleting job

gui/job_monitor.py:
 - keyPressEvent:
   Call .controller.set_changed(True)
Merge branch 'buildbot-nightly' into v2.1
Format and fix CLTools/README.md
Adds button to make CLTools module fail if $? != 0

This option supersedes 'return_code' which does not appear in the wizard
anyway.

It is on by default for new tools.
Markup
BuildBot FQDN
Updates notes
Merge branch 'buildbot-nightly-2.1' into 'v2.1'
Updates paths in dist/README.md
Merge pull request #958 from VisTrails/cltools-check-exitcode

This looks ok. return_code is for the uncommon case where you want to use non-zero exit code. It is not in the wizard because the interface get easily cluttered when adding too many options.
monitor->handle avoids confusion with JobMonitor

BaseMonitor->JobHandle
get_job_monitor()->get_job_handle()
Adds documentation to JobMixin
Style fixes
Fixes typo: params should be parameters
Fixes typo: there is no argument 'job_id'
More renaming of 'monitor' to 'handle'

There are enough "job monitor" things, this is a job handle.
Debug logging
Fixes typo in CLTools tests