Summary:ASTERISK-13892: [patch] Allow app_mp3 to play m3u playlist file
Reporter:Vincent Li (macli)Labels:
Date Opened:2009-04-03 11:48:00Date Closed:2009-12-04 14:27:39.000-0600
Versions:Frequency of
Environment:Attachments:( 0) app_mp3.diff
( 1) app_mp3.diff1
( 2) app_mp3.patch
( 3) m3u_issue.diff
Description:app_mp3 can only play one local single music and stream from http, this patch allow it to play local m3u playlist file.
Comments:By: Leif Madsen (lmadsen) 2009-04-09 10:44:15

License accepted! This looks like it could be useful :)

By: Leif Madsen (lmadsen) 2009-04-09 10:46:02

Don't forget to update the documentation either. If it is not already in XML format, that would also be an ideal change, but for now, you will need to update the docs in the code so that people at least know you can either pass it a file, or a playlist filename.


By: Vincent Li (macli) 2009-04-09 15:58:42

I added some documentation in XML format, second patch app_mp3.diff uploaded

I will add note here describing how to play m3u playlist file :-)

dialplan will look like this:
exten => 1234,1,Answer()
exten => 1234,n,MP3Player(/var/lib/asterisk/mp3/playlist.m3u)

and the playlist.m3u will be like this:


It is easy to generate mp3 playlist file with .m3u, do the following:

find /path/to/your-mp3-file -type file *.mp3 > playlist.m3u

By: Vincent Li (macli) 2009-04-09 16:11:41

the second patch seems have one single long line, I uploaded third patch app_mp3.diff1 to comform to code guide line :-)

By: Leif Madsen (lmadsen) 2009-06-16 14:01:31

This is a new feature, so it can only go into trunk, but I'm marking this as targeted for in the hopes we can get it merged in sooner rather than later. I will also set the status to Ready for Testing in the hopes we can get some other testers. Thanks!

By: David Vossel (dvossel) 2009-12-04 12:20:13.000-0600

macli, I don't understand some of your changes.  I uploaded a patch that strips out some some of the stuff that didn't seem necessary to me.  Let me know what you think.  The main thing I didn't understand was why you used the argument parser for a single argument and modified the http mpg321 code.  It's still your work of coarse and will be documented as such when it is committed.  

By: Vincent Li (macli) 2009-12-04 13:40:14.000-0600

Hi dvossel, I added the argument parser only for learning purpose, I do not remember exactly why I modifed mpg321 code, maybe it is related to some problem while playing some live online music list. Please go ahead using your simple solution, thanks!

By: Digium Subversion (svnbot) 2009-12-04 14:27:38.000-0600

Repository: asterisk
Revision: 233234

U   trunk/apps/app_mp3.c

r233234 | dvossel | 2009-12-04 14:27:38 -0600 (Fri, 04 Dec 2009) | 9 lines

.m3u support for Mp3Player app

(closes issue ASTERISK-13892)
Reported by: macli
     app_mp3.diff1 uploaded by macli (license )
Tested by: macli, dvossel