Summary: | ASTERISK-13892: [patch] Allow app_mp3 to play m3u playlist file | ||
Reporter: | Vincent Li (macli) | Labels: | |
Date Opened: | 2009-04-03 11:48:00 | Date Closed: | 2009-12-04 14:27:39.000-0600 |
Priority: | Major | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_mp3 |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
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. Thanks! 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: /var/lib/asterisk/mp3/1.mp3 /var/lib/asterisk/mp3/2.mp3 /var/lib/asterisk/mp3/3.mp3 .......... /var/lib/asterisk/mp3/n.mp3 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 1.6.3.0 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 Patches: app_mp3.diff1 uploaded by macli (license ) Tested by: macli, dvossel ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=233234 |