README: Fix broken link to user FAQ
Merge pull request #10 from Velmont/fix_broken_link

README: Fix broken link to user FAQ
Escape HTML characters in operation failure title and message

They are not escaped client-side, and although the title and message
is unlikely to ever contain content directly provided by the user,
it's typically still not supposed to be interpreted by HTML.

Also optionally disable the escaping of the message, in case the code
generating the message actually does generate HTML code.
Give useful error message when raising issue in closed review

The code just raised an exception, which got translated into a
standard "an unexpected error occurred" message being presented
to the user.  This is not an unexpected error, after all.
Give useful error message when creating comment via /showfile

If the commented file has been changed in the review (in any way
other than just being added or removed) the code for creating
comments can't deal with creating a comment in a single version
of the file rather than in the context of a diff.  Give a useful
error message when this limitation is hit, including a link to
a diff via which the comment could be created instead.

This is of course really just a stupid limitation that should be
fixed, but until then...
Improve client-side handling of failed asynchronous operations

If the client requests an asynchronous operation (i.e. provides a
callback function) and the operation fails, then make sure to call
the callback function after the error dialog is closed rather than
directly.  This makes it possible for the client code to try again
without risking trying again over and over while opening more and
more error dialogs on top of each other.
Fix installation of non-live extension versions

The path to 'tar' is spelled "configuration.executables.TAR", not
just "TAR".
Add <title> elements on some more pages

Was missing on /config, /branches and /tutorial (when not used to
show a specific tutorial, in which case the turorial's title was
set as the document's title automatically.)
Kill some unnecessary code duplication
Only return True in install("blah") if it actually installed something
Drop 2x assigned-but-not-read local variables 'all_ok'
Drop assigned-but-not-read local variable "started"
Fix two typos in README.md
Use the term "system group" instead of "Unix group"

(since the former is the terminology that install.py uses)
Drop assigned-but-not-read local variable "review_arg"
Merge pull request #13 from mo/minor_fixes_20121031

Minor fixes 20121031
Merge pull request #12 from mo/fix_bcrypt_restart_install_bug

Only return True in install("blah") if it actually installed something
Updated CONTRIBUTORS
Use --etc-dir argument if provider, otherwise ask for path

The etc_dir was effectively hard-coded to /etc/critic, for no good
reason.
Retrieve proper review id instead of using undefined local variable