Summary: | media-sound/timidity++-2.13.2-r10: distortion when buffer < 2048 frames | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | CLandel <titian> |
Component: | Current packages | Assignee: | Gentoo Sound Team <sound> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | miguelmarte |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | AMD64 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | Remove polling patch. |
Description
CLandel
2009-06-23 14:30:25 UTC
If you remove the ${FILESDIR}"/${P}-polling.patch from -r10, it works I assume? Miguel, your patch is about to get removed as it appears to break timidity++. CLandel, you need to verify this before it can be done. -r7 is now removed from tree. Sorry for being inactive for so long. How do I compile this without the patch? emerge won't let me when I edit the ebuild. Now with the r7 version removed, we need a fix, or the r7 version back. Has this behaviour been confirmed by anyone else? Created attachment 198955 [details]
Remove polling patch.
Same as -r10 without -polling.patch.
Or simply change these, "${FILESDIR}"/${P}-protos.patch \ "${FILESDIR}"/${P}-polling.patch To this, "${FILESDIR}"/${P}-protos.patch In -r10 ebuild and run `ebuild timidity++-2.13.2-r10.ebuild digest` or `emerge --digest timidity++` after it. (In reply to comment #3) > Sorry for being inactive for so long. > How do I compile this without the patch? emerge won't let me when I edit the > ebuild. Now with the r7 version removed, we need a fix, or the r7 version back. > Has this behaviour been confirmed by anyone else? > I haven't heard anyone reporting this before. The polling patch makes it not wake CPU so often... and I'm suspecting it might not be correct. So it's up to you to verify. I'm positive, the "polling" patch destroys the sound when the buffer is low. I commented the relevant stuff (the line with the patch and the backslash) out, but I needed the "--digest" option for emerge so I could verify that it would work. After emerging timidity++ with a hacked ebuild and --digest, it now works normally. Of course, I'm leaving this now like that on my system until r11 is out because I need low latency. Thank you for dealing with this. + 24 Jul 2009; Samuli Suominen <ssuominen@gentoo.org> + +timidity++-2.13.2-r11.ebuild, +files/conf.d.timidity.2, + +files/timidity.desktop.2: + LADSPA_PATH is now configurable from inside conf.d/timidity wrt #204713, + thanks to Adam Nielsen. Revert -polling.patch since it breaks realtime + playing with FluidR3 soundfont wrt #275198. Desktop entry now prefers GTK+ + frontend if available wrt #207311. |