remote/client: exit with 1 on KeyboardInterrupt

When concatenating commands with '&&' or using 'set -e', a keyboard
interrupt should lead to no further commands being executed.

To allow this, rather exit with 1 than 0 on keyboard interrupt.

Fixes: ef40183 ("remote/client: catch common errors and show a help message")
Signed-off-by: Bastian Krause <[email protected]>

Author: Bastian-Krause

CHANGES.rst

@@ -12,6 +12,7 @@ Bug fixes in 0.4.1
 - Fixed ``labgrid-client forward --remote``/``-R``, which used either the LOCAL
   part of ``--local``/``-L`` accidentally (if specified) or raised an
   UnboundLocalError.
+- Fix labgrid-client exit code on keyboard interrupt.
 
 Known issues in 0.4.1
 ~~~~~~~~~~~~~~~~~~~~~~~~~

labgrid/remote/client.py

@@ -1852,7 +1852,7 @@ def main():
                 print(f"{parser.prog}: error: {e}", file=sys.stderr)
             exitcode = 1
         except KeyboardInterrupt:
-            exitcode = 0
+            exitcode = 1
         except Exception:  # pylint: disable=broad-except
             traceback.print_exc()
             exitcode = 2