Checkbox

Merge lp://staging/~zyga/checkbox/remove-session-after-normal-exit into lp://staging/checkbox

  1. remove-session-after-normal-exit
  2. Merge into trunk
Proposed by Zygmunt Krynicki
Status: Work in progress
Proposed branch: lp://staging/~zyga/checkbox/remove-session-after-normal-exit
Merge into: lp://staging/checkbox
Diff against target: 11 lines (+1/-0)
1 file modified
plainbox/plainbox/impl/box.py (+1/-0)
To merge this branch: bzr merge lp://staging/~zyga/checkbox/remove-session-after-normal-exit
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Needs Fixing
Review via email: mp+151447@code.staging.launchpad.net

Description of the change

This branch removes the session file after all testing (and exporting is complete)

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

To work correctly, the session.clean() operation must be performed inside the "with:" statement.

Tested, it does nothing outside this block as the session is in a closed state.

review: Needs Fixing
Reply

Unmerged revisions

1959. By Zygmunt Krynicki

box: remove previous session after normal exit

Signed-off-by: Zygmunt Krynicki <email address hidden>

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches