Merge lp:~kissiel/checkbox/session-storage-renames into lp:checkbox
Status: | Merged |
---|---|
Approved by: | Chris Wayne |
Approved revision: | 4468 |
Merged at revision: | 4466 |
Proposed branch: | lp:~kissiel/checkbox/session-storage-renames |
Merge into: | lp:checkbox |
Diff against target: |
147 lines (+32/-8) 6 files modified
checkbox-ng/launchers/checkbox-cli (+7/-0) plainbox/plainbox/impl/runner.py (+1/-1) plainbox/plainbox/impl/session/assistant.py (+1/-1) plainbox/plainbox/impl/session/manager.py (+2/-2) plainbox/plainbox/impl/session/storage.py (+19/-3) plainbox/plainbox/impl/session/test_manager.py (+2/-1) |
To merge this branch: | bzr merge lp:~kissiel/checkbox/session-storage-renames |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Pierre Equoy | Approve | ||
Paul Larson | Approve | ||
Review via email: mp+302149@code.launchpad.net |
Description of the change
This MR brings changes to how sessions are stored in the filesystem.
Instead of using mkdtemp, the directory name that's used for storing session is now generated from two variables - session title and timestamp.
Title is determined by (in order of importance):
1) command line argument (--title)
2) basename of the launcher name
3) front-end name (e.g. checkbox-cli
To test it run checkbox-cli in different ways and observe ~/.config/
8a01902 plainbox:runner: fix typo in docstring
4573b73 plainbox:session: allow custom prefixes for session storage
265e26a plainbox:session: use timestamp in session storage path
4ed79a2 checkbox-ng: use launcher basename as sesion title
530faee checkbox-ng: add --title option for naming sessions
Apart from the small typo (see below), it looks good to me!
I tried to brutalize it by passing weird characters but it slugifies everything, so it's OK (well, Chinese words end up like a bunch of underscores but I guess the point is to be able to quickly name a session when testing, so it's not a big deal)