arnaudcoquelet / fusionpbx

Automatically exported from code.google.com/p/fusionpbx
0 stars 0 forks source link

No error on mp3 creation if lame not installed #789

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
I was attempting to set the system to record voicemails to mp3 instead of wav. 
mod_shout was running, the vm-message-ext parameter was set to mp3 in the 
voicemail.conf.xml, but it was still just recording to wav. No errors in the 
cli/log. 

Went through the voicemail scripts and found the section in 
/scripts/app/voicemail/resources/functions/record_message.lua: 

--convert the wav to an mp3 
--apt-get install lame 
resample = "/usr/bin/lame -b 32 --resample 8 -m s 
"..voicemail_dir.."/"..voicemail_id.."/msg_"..uuid..".wav 
"..voicemail_dir.."/"..voicemail_id.."/msg_"..uuid..".mp3"; 
session:execute("system", resample); 

It turned out lame was not installed on my system. However, a message in the 
logs in the above chunk of code would be good for troubleshooting. Also, the 
above code would not handle non-standard install locations. So perhaps a "which 
lame", which inserts the location into the resample command, or logs an error 
if it's not found?

Running version 3.6.3 on Debian 7.7 (found it on Ubuntu as well).

Original issue reported on code.google.com by teleut...@gmail.com on 9 Dec 2014 at 3:53