Bug 14338 - Tmux -c (CWD) requires Tmux 1.9
Summary: Tmux -c (CWD) requires Tmux 1.9
Status: RESOLVED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: oe-core other (show other bugs)
Version: unspecified
Hardware: All Multiple
: Medium+ normal
Target Milestone: 3.4 M1
Assignee: Peter Budny
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2021-04-05 19:43 UTC by Peter Budny
Modified: 2021-04-22 15:21 UTC (History)
2 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Budny 2021-04-05 19:43:34 UTC 
OE-Core rev 938a089b52e7bf5f67406f55234ab2d22236e9b6, which was brought into Poky in rev 404f7cec87bc9fabe48a5dbff38690faf96094b3, effectively changes the minimum version of Tmux that's required. `tmux new -c <cwd>` requires tmux >= 1.9.

The version of Tmux is being checked in one class (TmuxRunning, which requires it for a particular pane command) but not in the others (TmuxNewWindow and Tmux). With this change, all of these classes require Tmux >=1.9.

(I've opened a ticket with my sysadmins to try to get Tmux upgraded, as these are both ancient versions... Tmux 1.9 is from February 2014, and Tmux 1.8 is from March 2013.  But it's still a bug in OE-Core that it executes this command without checking the version of Tmux, or otherwise conveying this requirement to the user.)

The other classes that use `tmux new -c` should check the version of Tmux. If it's <1.9, they should either error or they should run Tmux without the `-c <cwd>`, as they did before.
Comment 1 Randy MacLeod 2021-04-08 14:38:09 UTC 
Are you able to submit a patch?
Comment 2 Peter Budny 2021-04-16 17:04:11 UTC 
The patch turned out to be pretty easy. I've sent it to the oe-core mailing list (which I see someone has already noticed).
Comment 3 Randy MacLeod 2021-04-16 23:44:54 UTC 
That's great, thanks so much Peter! ../Randy