Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 117345

Summary: cplay shows title when run inside xterm or aterm, but not inside screen
Product: Gentoo Linux Reporter: Denilson Sá Maia <denilsonsa>
Component: Current packagesAssignee: Gentoo Sound Team <sound>
Status: RESOLVED UPSTREAM    
Severity: normal CC: dickey
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: cplay-1.49_term-title.patch

Description Denilson Sá Maia 2006-01-01 06:05:19 UTC
I guess the best way to describe this is telling how to reproduce this bug.
1. Open xterm (or aterm, I guess rxvt is fine too).
2. Run cplay
3. Notice the title is changed to "cplay". If you play any file, the title will change to display current file.
4. Close cplay (press Q). Run screen.
5. Run cplay (now inside screen).
6. Notice there is no title update when inside screen.

If someone thinks screen does not support title updates, you should try to run emerge inside screen.

(I think I will make a patch to fix this)
Comment 1 Denilson Sá Maia 2006-01-01 06:23:41 UTC
Created attachment 75910 [details, diff]
cplay-1.49_term-title.patch

I changed it. Now cplay does support lots of terminals to set title. Most of the changes were inspired by /usr/lib/portage/pym/output.py. I still don't know why portage uses "ESC]2;", while cplay uses "ESC]0;'.

Finally, when running cplay inside screen inside aterm, the title is not set, but attaching this screen to a xterm works. I guess this is a bug with screen and aterm.

Please test the patch, take a look at it and tell if it works, or how it can be improved.
Comment 2 Thomas Dickey 2006-01-02 13:17:07 UTC
This is a bug in screen.  See
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=344759
Comment 3 Denilson Sá Maia 2006-01-04 07:32:44 UTC
Ok, so we have one bug in screen (about it not working with aterm) and one bug with cplay.

The bug with cplay should be fixed with my patch. After reading the link you sent, I think we should also change cplay to use ESC]2; instead of ESC]0;. Is this correct? Should we change this too?

Anyway, I think this patch should be added to portage, and then fowarded upstream.
Comment 4 Diego Elio Pettenò (RETIRED) gentoo-dev 2006-10-04 23:46:57 UTC
This is a minor problem, send it upstream first.