[Bug 752669] New: games/openttd-openmsx: requires timidity
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c0 Summary: games/openttd-openmsx: requires timidity Classification: openSUSE Product: openSUSE.org Version: unspecified Platform: i586 OS/Version: openSUSE 12.1 Status: NEW Severity: Normal Priority: P5 - None Component: 3rd party software AssignedTo: suse@ammler.ch ReportedBy: matthias@mailaender.name QAContact: opensuse-communityscreening@forge.provo.novell.com CC: christian.morales.vega@gmail.com, prusnak@suse.com Found By: --- Blocker: --- Why do you need Timidity for the openttd music package? I searched the README file and did not find a reference. You can also download the music files ingame and I don't think that openTTD also downloads timidity, but uses it's own midi player. http://qa.debian.org/cgi-bin/dcontrol?package=openttd-openmsx does not depend on timidity at all. http://qa.debian.org/cgi-bin/dcontrol?package=openttd only recommends it. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c1 --- Comment #1 from Marcel Gmür <suse@ammler.ch> 2012-03-19 17:08:18 UTC --- I guess, we could also add timidity via recommends, if that fits better for you. That you can download openmsx via ingame download service is the reason, I submitted openmsx also to Factory so you get it installed via recommendation. Else you would download it and not be able to listen as you miss the music player. If I may ask, which other midi player is available from Factory (or any other repo)? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c2 --- Comment #2 from Marcel Gmür <suse@ammler.ch> 2012-03-19 17:10:05 UTC --- Also you referenced to openttd in debian recommends it, this is the same here. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c3 --- Comment #3 from Cristian Morales Vega <christian.morales.vega@gmail.com> 2012-03-19 17:54:08 UTC --- The bug report is valid. I don't know the first thing about OpenTTD, I never played it. But if you aren't able to listen the music it must be because of checking libtimidity... not found in the **main** openttd package. A "BuildRequires: libtimidity-devel" should fix the problem. I guess fluidsynth is a better option than timidity (at least there is a released version from this decade), but the "midi player" will not be something you can select. The devs of OpenTTD probably used libtimidity years ago and now they wouldn't see a cause to change the code to use fluidsynth. I guess timidity needs the equivalent of a soundfount2 file. That's probably why debian recommends timidity in the **main** package. Or perhaps it is loaded at runtime and "timidity" is the name of the package that contains the library. No idea, I'm not going to check the Debian package. Whatever the reason: a) openttd-openmsx must not require timidity b) openttd should build against libtimidity c) perhaps "b" is not enough to make the audio work -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c4 --- Comment #4 from Matthias Mailänder <matthias@mailaender.name> 2012-03-19 20:02:32 UTC --- It is a little ugly to have an unwanted program with a pixelish .desktop icon (that's how I found out) as a dependency. https://build.opensuse.org/package/rawlog?arch=i586&package=openttd&project=games&repository=openSUSE_12.1 says checking libtimidity... not found and there is a https://bitbucket.org/OpenTTD/openttd/src/d62b1d0ea0ba/src/music/libtimidity... Debian does not have http://packages.debian.org/search?searchon=names&keywords=libtimidity but http://software.opensuse.org/search?q=libtimidity-devel&baseproject=ALL&lang=de I don't understand why http://wiki.openttd.org/Compiling_on_Linux insists of installing timidity++ instead of the library and why the library is not mentioned as a optional dependency in http://svn.openttd.org/trunk/readme.txt When I build with BuildRequires: libtimidity-devel and sudo zypper remove timidity I don't get music. This is confusing. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c5 --- Comment #5 from Marcel Gmür <suse@ammler.ch> 2012-03-19 21:03:02 UTC --- configure at openttd checks for timidity as possible sound driver (openttd-opensfx), but not for the music, there it still needs a "external" midi player, which does not need to be timidity, that is why debian only recommends it. As said, changing it to recommends is no big deal, but you still would need to install a music player for openttd-openmsx -> so changing tag of timidity from requires to recommends solves this ticket? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c Marcel Gmür <suse@ammler.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c6 --- Comment #6 from Marcel Gmür <suse@ammler.ch> 2012-03-19 21:31:42 UTC --- you submitted #110060 but in your last reply here, you write that it didn't work, so why add timidity for openttd? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c7 --- Comment #7 from Marcel Gmür <suse@ammler.ch> 2012-03-19 22:10:06 UTC --- In the meantime, I "fixed" openttd-openmsx and changed the tag to recommends which also superseded the factory sr#109786 (new sr: #110068) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c8 Marcel Gmür <suse@ammler.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED --- Comment #8 from Marcel Gmür <suse@ammler.ch> 2012-03-19 22:20:21 UTC --- As this ticket is about openttd-openmsx and I changed the spec, we can close this ticket, I am not happy about your change to openttd, please review that. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c9 --- Comment #9 from Bernhard Wiedemann <bwiedemann@suse.com> 2012-03-20 00:00:09 CET --- This is an autogenerated message for OBS integration: This bug (752669) was mentioned in https://build.opensuse.org/request/show/110068 Factory / openttd-openmsx -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c10 --- Comment #10 from Cristian Morales Vega <christian.morales.vega@gmail.com> 2012-03-19 23:39:58 UTC --- I checked it more deeply. timidity/libtimidity is clearly to play the music. List of music drivers: libtimidity: LibTimidity MIDI Driver extmidi: External MIDI Driver null: Null Music Driver List of sound drivers: sdl: SDL Sound Driver null: Null Sound Driver But in any case the libtimidity backend in this game, even if it has a higher priority than the "extmidi" backend (that uses the timidity binary) is broken and the music just fails to play with it without giving any error message. So it's better to avoid the libtimidity-devel BuildRequire... The test programs in the libtimidity tarball work fine, so the problem isn't in libtimidity itself. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c11 --- Comment #11 from Marcel Gmür <suse@ammler.ch> 2012-03-19 23:53:37 UTC --- ok, thanks for testing Feel free to report it to upstream: http://bugs.openttd.org -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c12 --- Comment #12 from Marcel Gmür <suse@ammler.ch> 2012-03-19 23:56:22 UTC --- what could be discussed it moving the "Recommends: timidity" from openttd-openmsx to openttd like in debin -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c13 --- Comment #13 from Cristian Morales Vega <christian.morales.vega@gmail.com> 2012-03-20 00:26:09 UTC --- I don't think it makes a lot of sense to report upstream. I looked at the code and it doesn't work... because the code is only complete for the PSP. For anything else is not even supposed to work. Given the ugly situation (play music using an external binary, really???) I don't think the "Requires" you had originally is wrong at all. So, as you prefer... -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c14 --- Comment #14 from Marcel Gmür <suse@ammler.ch> 2012-03-20 08:18:45 UTC --- C.M. Vega mentioned another player: fluidsynth, that was the main reason to change it -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c15 --- Comment #15 from Cristian Morales Vega <christian.morales.vega@gmail.com> 2012-03-20 10:21:40 UTC --- And in theory fluidsynth could be used. But after looking at the code I saw timidity is hardcoded. So you can use whatever player you want, but you have to select one when you compile openttd and it can no be changed after that (timidity being the default) I will try to look into completing libtimidity/adding libfluidsynth support this weekend. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c16 --- Comment #16 from Marcel Gmür <suse@ammler.ch> 2012-03-20 14:13:24 UTC --- well, I meant as external midi player..., so installing timidity is just recommended -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c17 --- Comment #17 from Marcel Gmür <suse@ammler.ch> 2012-03-20 14:15:09 UTC --- according to upstream, getting libtimidity working with openttd is not possible... -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=752669 https://bugzilla.novell.com/show_bug.cgi?id=752669#c18 --- Comment #18 from Matthias Mailänder <matthias@mailaender.name> 2012-03-20 18:22:17 UTC --- My bad, I revoked my OBS request and opened a feature request upstream: http://bugs.openttd.org/task/5115 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply@novell.com