Tool: PEP8 Style Checker
Processed Files:
    rbtools/commands/land.py



Tool: Pyflakes
Processed Files:
    rbtools/commands/land.py

This isn't quite the right fix, though it's close.
We want to keep this test for AttributeError, since it's handling the specific case of the server being old and not having the approved or approval_failure attributes.
Instead, move the code from the finally block out of finally so that it runs after the whole try/except block and before the if is_local check. That way any uncaught exceptions from the try will just raise up further and be handled by the top-level logger.

Also, please see https://www.reviewboard.org/docs/codebase/dev/writing-good-descriptions/ for tips on formatting your summary/description/testing done.

Tool: Pyflakes
Processed Files:
    rbtools/commands/land.py



Tool: PEP8 Style Checker
Processed Files:
    rbtools/commands/land.py

Your summary needs work (no abbreviations like "b4", proper grammar/capitalization). The description should also explain what the problem was, not just your solution to it.
Finally, please add the bug number into the "Bugs" field