Merge lp:~sylvain-pineau/checkbox/fix-1458577 into lp:checkbox

[Sylvain Pineau]

Fixes the linked bug

[Zygmunt Krynicki (zyga)]

11:57 <@zyga> spineau: that bug is more complex
11:57 <@zyga> spineau: +1 on the merge but there's a different problem there too
11:57 <@zyga> spineau: you can Ctrl-C at so many places after entering input and before we have a result
11:57 <@zyga> spineau: that we still lose stuff
11:57 <@zyga> spineau: the only way to solve that would to instantly keep a partial "under construction" result in the session somewhere
11:58 <@spineau> zyga: I see

review: Approve

[Sylvain Pineau (sylvain-pineau)]

Let's land this simple fix first.

=== modified file 'plainbox/plainbox/impl/commands/inv_run.py'
--- plainbox/plainbox/impl/commands/inv_run.py	2015-11-04 14:35:46 +0000
+++ plainbox/plainbox/impl/commands/inv_run.py	2016-02-05 10:49:02 +0000
@@ -807,7 +807,7 @@
                     if cmd == 'run' or cmd is None:
                         result_builder = self.runner.run_job(
                             job, job_state, self.config, ui
-                        ).get_builder()
+                        ).get_builder(comments=comments)
                     elif cmd == 'comment':
                         new_comment = input(self.C.BLUE(
                             _('Please enter your comments:') + '\n'))