[asterisk-dev] [Code Review] Patch: Adds 'astsounddir' configuration option to asterisk.conf.

[Tzafrir Cohen]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1080/#review3107
-----------------------------------------------------------


Why not set 'astdatadir' instead?

- Tzafrir


On 2011-01-19 03:34:16, lottc wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1080/
> -----------------------------------------------------------
> 
> (Updated 2011-01-19 03:34:16)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> The patch adds the 'astsounddir' option to /etc/asterisk/asterisk.conf.
> This allows you to override Asterisk's sound directory without needing to
> make symlinks.  A practical application of this would be to load the contents
> of /var/lib/asterisk/sounds into a tmpfs or ramdisk mount.  Asterisk will then
> read the files directly from memory (provided tmpfs hasn't swapped) instead of
> a calling out to disk for every file.
> 
> This patch also adds the Sounds directory location to 'core show settings'.
> 
> 
> Diffs
> -----
> 
>   /trunk/Makefile 302411 
>   /trunk/apps/app_dial.c 302411 
>   /trunk/build_tools/make_defaults_h 302411 
>   /trunk/include/asterisk/paths.h 302411 
>   /trunk/main/app.c 302411 
>   /trunk/main/asterisk.c 302411 
>   /trunk/main/file.c 302411 
> 
> Diff: https://reviewboard.asterisk.org/r/1080/diff
> 
> 
> Testing
> -------
> 
> Compiled.
> Verified that setting astsounddir overrides default with 'core show settings'.
> Tested playback of sounds.
> Has been running on about a dozen systems for 1+ month.
> 
> 
> Thanks,
> 
> lottc
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110119/af840ce4/attachment.htm>