70592 2004-11-09 09:52 0000 mpg123 0.59s-r6 makes tty malfunction, requiring a reset 2005-08-13 05:42:26 0000 1 1 1 Unclassified Gentoo Linux Applications unspecified x86 Linux RESOLVED FIXED P2 normal --- 1 veneroso_dc@hotmail.com sound@gentoo.org kero@matfyz.cz veneroso_dc@hotmail.com 2004-11-09 09:52:25 0000 When starting mpg123 like this: mpg123 --list playlist.m3u --shuffle --random --control Upon exiting mpg123, the terminal does not echo back typed text or command history. The only remedy that I know of is to type 'reset'. This behaviour only seems to become apparent when run with the --control option. If q is pressed while playing the first song, then it exits to the terminal normailly, but if you first press f (or even ctrl-c) to skip to the next song, then press q the terminal malfunctions. It also malfunctions if its allowed to progress to the next song normally and then exit by pressing q. veneroso_dc@hotmail.com 2005-01-29 15:34:56 0000 Well, after no replys, this bug either cannot be fixed, or cannot be verified. I only discovered it because my monitor failed and my only recorse was to access my box via a remote terminal. It was an older machine, but I still wanted to be able to hear my music while I was forced to use the other one. I have since purchased a new monitor, and this is of little priority to me. I did notice that there is a new version of this package, and I have decided to test it. Below are my results: media-sound/mpg123-0.59s-r9 Same problems, reset must be executed to return to normal operation. I can remove most of the headaches if i add && reset at the end of the command line. kero@matfyz.cz 2005-05-30 11:58:41 0000 *** Bug 94494 has been marked as a duplicate of this bug. *** balrogg@gmail.com 2005-06-10 07:11:41 0000 Created an attachment (id=60986) Patch to avoid terminal line settings be set more than once (In reply to comment #0) This is because in -C mode term_init() is called everytime a new songs starts playing, so the first run of term_init() saves the original terminal line properties in a variable but when playing a second song the value gets overwritten. It also results in registering term_restore() with atexit() as many times as songs were played, but term_restore() has a check that avoids executing it more than once. I attached a patch that fixes it by adding the same check to term_init(). balrogg@gmail.com 2005-08-07 19:02:44 0000 Created an attachment (id=65376) Plain-text of the above patch I realized it shouldn't be gzipped so i'm posting the plain-text version of the patch. It would be good if someone added it to the ebuild cause the bug can be really annoying and it's easy to verify, although not many people use the -C option. chainsaw@gentoo.org 2005-08-13 05:42:26 0000 This is in portage; Andrzej, thanks for your work. 60986 2005-06-10 07:11 0000 Patch to avoid terminal line settings be set more than once mpg123-pre0.59sbalrog.patch.gz text/plain H4sICNmEqEIAA21wZzEyMy1wcmUwLjU5c2JhbHJvZy5wYXRjaABlUk1z0zAQPaNf8bgwzjg2jvNF k8Kkw50TJy6MbK8dgS15pE3TlP54VqlhWtBBo13te/v2SUx++G6s4WQGE1DrvqcGdE/+wmYgaFg6 IzjbIbD2HDD2+mIkNBbZZwyuIRUcnO0v0C2TBx8JrfGBn2GtsIcjhTl0ELpgGgK1LdV8rXTedMbq Xgl0iAf0xhJG70bybCjgTJ7QEXPs6kTZ2RtmstC2iRSeWicVsUxZx/AUWBJy5151wL3uT5Tjrhe9 /2uJRkzQZKbO+q8Zchq0vSD6EWIUx5pkRTOokdEY9GA4x1dpOGquj6o1D1LOR3G1uqCf9EdBkosE lfR+oPrE0uNqn7M15erONv6RfuCbrpx35/DT4LbSvXfd4TGmcue7T0o1pm2RfdEnj2HsFuXyfZwg r6eomkKVZdnrgjdlURTZosiWBRbb3Wq9W6zy4s9CGneVpuk/RBG2zopNVtxE2LrcrW7y7Wa1WJbl ZjXBDgdk5Xq+RSr7BxwOCr8UIF/HnyaPjRNTjNsrqAzPrpPVVU/4iGKvUsC0SF7mn57wdtReD/k1 W7PvZ7EO8tJ88jZSSaA5vkDy8h1n+3hh2oRr+T+a2SfF/J10n+EWxUy0/QaPvlP+/wIAAA== 65376 2005-08-07 19:02 0000 Plain-text of the above patch mpg123-pre0.59sbalrog.patch text/plain dGVybV9pbml0KCkgaXMgY2FsbGVkIGV2ZXJ5dGltZSBhIG5ldyBzb25nIHN0YXJ0cyBwbGF5aW5n IGluIC1DIG1vZGUKc28gb25seSBhZnRlciB0aGUgZmlyc3Qgc29uZyBmaW5pc2hlcywgYXMgYSBz aWRlIGVmZmVjdCB0aGUgb3JpZ2luYWwKdGVybWluYWwgbGluZSBwcm9wZXJ0aWVzIHdlcmUgZ2V0 dGluZyBvdmVyd3JpdHRlbiBhbmQgdGhlcmVmb3JlIHdlcmUKbm90IHJlc3RvcmVkIHRvIHRoZSBv cmlnaW5hbCB2YWx1ZS4gQWxzbyBhcyBhIHNpZGUgZWZmZWN0IHRlcm1fcmVzdG9yZSgpCndhcyBj YWxsZWQgYXMgbWFueSB0aW1lcyBhcyBzb25ncyB3ZXJlIHBsYXllZCwgYXQgZXhpdC4gVGhlIHBh dGNoCmZpeGVzIHRoaXMgYnkgbGV0dGluZyB0aGUgdGhpbmdzIGJlIGV4ZWN1dGVkIG9ubHkgb25j ZS4KQW5kcnplaiBaYWJvcm93c2tpIDxiYWxyb2dAemFib3Iub3JnPgoKZGlmZiAtTmF1ciBtcGcx MjMvdGVybS5jIG1wZzEyM2IvdGVybS5jCi0tLSBtcGcxMjMvdGVybS5jCTIwMDAtMTAtMzAgMTc6 NDU6MTQuMDAwMDAwMDAwICswMDAwCisrKyBtcGcxMjNiL3Rlcm0uYwkyMDA1LTA2LTA5IDE3OjUy OjQ5Ljc2NDEzMjI2NCArMDAwMApAQCAtMjUsNyArMjUsOCBAQAogewogICBzdHJ1Y3QgdGVybWlv cyB0aW87CiAKLSAgdGVybV9lbmFibGUgPSAwOworICBpZiAodGVybV9lbmFibGUgfHwgIXBhcmFt LnRlcm1fY3RybCkKKyAgICByZXR1cm47CiAKICAgYXRleGl0KHRlcm1fcmVzdG9yZSk7CiAgIGlm KHRjZ2V0YXR0cigwLCZ0aW8pIDwgMCkgewo=