Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 330039 - app-misc/tmux-1.3: ewarn users of deprecated commands
Summary: app-misc/tmux-1.3: ewarn users of deprecated commands
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: High enhancement
Assignee: Alex Alexander (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-27 09:34 UTC by Florian CROUZAT (SuperFlo)
Modified: 2010-08-19 18:12 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian CROUZAT (SuperFlo) 2010-07-27 09:34:01 UTC
Regarding bug 328963 I think we should add a einfo/ewarn to the ebuild in the pkg_postinst section to inform tmux users they should check their conf because a few commands have been removed and cause tmux to output warnings/errors.
My english isn't really good so I won't write the real message but maybe something like this:

ewarn "Commands up-pane (upp), down-pane (downp) and select-prompt have been replaced by a new syntax."
ewarn "You might want to check your configuration file and update it according to the 1.3 changes.
ewarn "See http://tmux.cvs.sourceforge.net/viewvc/tmux/tmux/CHANGES for detailed informations."

Also note that tmux.vim hasn't been updated accordingly to the modifications, I've sent a patch upstream so I don't know if we should wait it's getting applied or apply if ourselves, doesn't look really critical, though.
Comment 1 Johan Bergström 2010-07-27 20:09:45 UTC
It might also be nice to warn users that a newer client won't work against an older server or vice versa.
Comment 2 Florian CROUZAT (SuperFlo) 2010-07-28 08:00:33 UTC
I agree. In a pkg_preinst section then so that they can press ctrl-c.
Comment 3 Alex Alexander (RETIRED) gentoo-dev 2010-08-19 17:14:03 UTC
fixed, I added a pretty thorough warning in pkg_setup :)

thanks for reporting!
Comment 4 Florian CROUZAT (SuperFlo) 2010-08-19 17:58:33 UTC
Thanks for the ewarn commit.
But, it has a typo. Not sure I should have reopened the bug for this, so I just say it: "You'll have to use an existing client to to end your old session" ; there is two "to" in there ;)
Comment 5 Alex Alexander (RETIRED) gentoo-dev 2010-08-19 18:12:34 UTC
nice catch. fixed :)