Merge lp:~hyperair/byobu/fix-reuse-sessions-inverted-logic into lp:byobu

Proposed by Chow Loong Jin
Status: Merged
Merge reported by: Dustin Kirkland 
Merged at revision: not available
Proposed branch: lp:~hyperair/byobu/fix-reuse-sessions-inverted-logic
Merge into: lp:byobu
Diff against target: 14 lines (+2/-2)
1 file modified
usr/lib/byobu/include/select-session.py (+2/-2)
To merge this branch: bzr merge lp:~hyperair/byobu/fix-reuse-sessions-inverted-logic
Reviewer Review Type Date Requested Status
Dustin Kirkland  Approve
Review via email: mp+384507@code.launchpad.net

Description of the change

The reuse_sessions logic is inverted -- I have ~/.byobu/.reuse-session present, but byobu's select-sessions.py is calling tmux with "new-session .... set-option destroy-unattached". This merge request fixes that logic.

To post a comment you must log in.
Revision history for this message
Dustin Kirkland  (kirkland) :
review: Approve
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Thanks. Merged and pushed to https://github.com/dustinkirkland/byobu

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'usr/lib/byobu/include/select-session.py'
--- usr/lib/byobu/include/select-session.py 2020-02-09 16:35:32 +0000
+++ usr/lib/byobu/include/select-session.py 2020-05-25 15:30:37 +0000
@@ -133,9 +133,9 @@
133 # must use the binary, not the wrapper!133 # must use the binary, not the wrapper!
134 if backend == "tmux":134 if backend == "tmux":
135 if reuse_sessions:135 if reuse_sessions:
136 os.execvp("tmux", ["tmux", "-u", "attach", "-t", session_name])
137 else:
136 os.execvp("tmux", ["tmux", "-u", "new-session", "-t", session_name, ";", "set-option", "destroy-unattached"])138 os.execvp("tmux", ["tmux", "-u", "new-session", "-t", session_name, ";", "set-option", "destroy-unattached"])
137 else:
138 os.execvp("tmux", ["tmux", "-u", "attach", "-t", session_name])
139 else:139 else:
140 os.execvp("screen", ["screen", "-AOxRR", session_name])140 os.execvp("screen", ["screen", "-AOxRR", session_name])
141141

Subscribers

People subscribed via source and target branches