Bug 538224 - fluidsynth: PulseAudio support
Summary: fluidsynth: PulseAudio support
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: fluidsynth
Version: 12
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Kevin Kofler
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-11-17 23:46 UTC by Michael Monreal
Modified: 2010-01-21 00:15 UTC (History)
2 users (show)

Fixed In Version: 1.0.9-5.fc12
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-01-21 00:10:00 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Michael Monreal 2009-11-17 23:46:24 UTC
FluidSynth supports PulseAudio since 1.0.9 (which is currently shipped for F11 and F12 bud sadly the Fedora package does not include this support. All that has to be done may be adding pulseaudio-devel to the BuildRequires?

As PulseAudio is now deeply integrated with Fedora, supporting it in FluidSynth makes sense IMHO.

Comment 1 Orcan Ogetbil 2009-11-18 02:47:49 UTC
This has been recently discussed in bug 500087 from comment #12 on.
Also on a recent FESCo meeting.

In my opinion, adding a single BR is not a solution. Indeed it will be a new source of problems. Fluidsynth is a quite mature library and is widely used in MIDI applications which are typically used for production. Linking it to the buggy pulseaudio, which only works for a handful of people, and which is fundamentally flawed to be used for production, has a great potential of opening a door of bug reports. And I'm not willing to support it.

Nevertheless, FESCo's decision was to add pulseaudio support and I respect their decisions. I am assigning the bug to the person they chose for pulseaudio support. 

I am not going to add the BR myself. History will not write such a shame on my book :)

Comment 2 Kevin Kofler 2009-11-18 08:19:20 UTC
For reference: https://fedorahosted.org/fesco/ticket/265

Comment 3 Kevin Kofler 2009-11-18 08:26:15 UTC
I requested watchbugzilla so I can handle any PA-related bug reports (as discussed in the FESCo meeting).

Comment 4 Michael Monreal 2009-11-18 08:30:48 UTC
(In reply to comment #1)
> [..] Linking it to the buggy pulseaudio, which only works for a 
> handful of people, and which is fundamentally flawed to be used 
> for production [..]

All I can say is that my experience with pulseaudio is a totally different one.
It works, it rocks... and the only problems *I* run into when having software
that does not support it...

Thanks to Lennart, audio on Linux is finally stopping to be a pain!

Comment 5 Orcan Ogetbil 2009-11-18 08:33:44 UTC
Requests approved for Kevin.

Michael, I'm happy for you. Good luck.

Comment 6 Michael Monreal 2009-11-18 11:06:33 UTC
(In reply to comment #3)
> I requested watchbugzilla so I can handle any PA-related bug reports 

Any time estimation for pulse-enabled packages? Also, will this be F12 only or will F11 also get the fix?

Comment 7 Kevin Kofler 2009-12-09 08:18:35 UTC
Sorry for the time elapsed. I'm enabling this in F13 Rawhide now, but I'm also planning to push updates for F11 and F12.

Comment 8 Orcan Ogetbil 2009-12-10 05:48:41 UTC
A year from now, I will ask you folks how many midi files you have listened, just for the fun of it, with the kmid-qsynth-fluidsynth-pulseaudio combination.

Why will I ask? Just for the fun of it :)

Comment 9 Michael Monreal 2010-01-06 21:42:27 UTC
I noticed there is a PA enabled build for F13 now. Will this also be changed for F12 or not?

Comment 10 Kevin Kofler 2010-01-08 19:06:29 UTC
Yes, as there have apparently been zero bugs filed about the Rawhide change, so I guess it's working well. I've submitted F12 and F11 builds and will be queueing them for updates-testing.

Comment 11 Fedora Update System 2010-01-08 19:24:35 UTC
fluidsynth-1.0.9-5.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/fluidsynth-1.0.9-5.fc12

Comment 12 Fedora Update System 2010-01-08 19:24:52 UTC
fluidsynth-1.0.9-5.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/fluidsynth-1.0.9-5.fc11

Comment 13 Fedora Update System 2010-01-12 23:51:41 UTC
fluidsynth-1.0.9-5.fc12 has been pushed to the Fedora 12 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update fluidsynth'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2010-0432

Comment 14 Kevin Kofler 2010-01-15 02:22:26 UTC
From my own testing:
The quality is good enough for casual KMid usage with these settings:
synth.sample-rate = 22050
audio.period-size = 256
audio.periods = 16
Faster machines than this old P4 Northwood might perform better, YMMV.

That said, there's a lot of stuff which could probably be done to improve quality upstream: the larger period-size, the more irregular timing becomes, the shorter period-size, the more noise gets produced, 256 is the most tolerable compromise.

Oh, and the KMid snapshot currently in Fedora also has some bug where it can send junk to the sequencer which crashes FluidSynth, but as it screws up similarly with timidity++, I'll primarily blame KMid for that (though crashing is IMHO always a bug). I need to test the recent official kmid2 tarball (but now I have 2 sequencers to test with, always good if you need to know where to put the blame).

Comment 15 Fedora Update System 2010-01-21 00:09:55 UTC
fluidsynth-1.0.9-5.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 16 Fedora Update System 2010-01-21 00:15:14 UTC
fluidsynth-1.0.9-5.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.