patchGNU Octave - Patches: patch #7530, Add a PulseAudio...

 
 

patch #7530: Add a PulseAudio "backend" for playaudio.

Submitter:  Fabian Deutsch <fabiand>
Submitted:  Tue 12 Apr 2011 03:37:41 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  None Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 29 Jul 2011 04:35:40 PM UTC, comment #5: 
Jordi Gutiérrez Hermoso <jordigh>
Group Member
Fri 29 Jul 2011 03:56:18 PM UTC, comment #4: 

Could I possibly persuade any of you guys to apply this patch as my build environment is currently quite messy?

Thanks
Søren

Søren Hauberg <hauberg>
Fri 29 Jul 2011 02:32:47 PM UTC, comment #3: 

Okay, I see your point. Go ahead and apply the patch.

Jordi Gutiérrez Hermoso <jordigh>
Group Member
Fri 29 Jul 2011 07:07:33 AM UTC, comment #2: 

The function in question (playaudio) is part of core Octave and not Octave-Forge, so I am re-opening this here.

While I agree with Jordi that it would be better in the long run to use some cross-platform audio library, I think we have to be pragmatic with this one. The current implementation is highly Unix specific; so specific that it does not appear to work on Linux anymore. So while the suggested patch is not the perfect solution it does improve on the current situation.

I think we should apply the patch.

Søren Hauberg <hauberg>
Tue 12 Apr 2011 04:41:20 PM UTC, comment #1: 

The audio package is in Octave-Forge. I know that to users the administrative separation between Octave and Octave-Forge appears very artificial, and it does to me too. I do want to bring the two projects much closer administratively, but please bear with us in the meantime and take this to the Octave-Forge website.

A true cross-platform solution to audio is probably to use SDL instead of what the audio package currently does.

Jordi Gutiérrez Hermoso <jordigh>
Group Member
Tue 12 Apr 2011 03:37:41 PM UTC, original submission:  

Most modern Linux distributions provide packages for PulseAudio and use it as a default sound-system (Fedora, Ubuntu, openSUSE, ...). Even more some distributions (like Fedora) don't allow users to write to /dev/dsp which is used by playaudio.

Therefor the attached patch changes touches playaudio.m and adds suport for PulseAudio using it's paplay utility. But paplay is just choosen if the classic way ("cat %s > /dev/dsp") fails.

This patch was tested on Fedora.

Fabian Deutsch <fabiand>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by hauberg (Posted a comment)
  • -email is unavailable- added by jordigh (Posted a comment)
  • -email is unavailable- added by fabiand (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-07-29 jordigh StatusInvalid Done
        Open/ClosedOpen Closed
    2011-07-29 hauberg Open/ClosedClosed Open
    2011-04-12 jordigh StatusNone Invalid
        Open/ClosedOpen Closed
    2011-04-12 fabiand Attached File- Added octave-playaudio-with-pulseaudio.patch, #23228

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code