| Summary: | ASTERISK-17139: [PATCH] Adds astsounddir configuration option. | ||
| Reporter: | Lott Caskey (lottc) | Labels: | patch |
| Date Opened: | 2010-12-20 23:20:27.000-0600 | Date Closed: | |
| Priority: | Major | Regression? | No |
| Status: | Open/New | Components: | Core/Configuration |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) asterisk-1.6.2.16rc1-astsounddir.patch ( 1) asterisk-trunk_302411-astsounddir.patch ( 2) asterisk-trunk_321560-astsounddir.patch | |
| Description: | I found myself needing to modify the location of the sounds directory out from /var/lib/asterisk/sounds. This patch allows you to define astsounddir in asterisk.conf. | ||
| Comments: | By: Paul Belanger (pabelanger) 2010-12-21 17:56:02.000-0600 Any new features must be against trunk. Additionally, I'd recommend starting a discussion on the -dev mailing list. By: Leif Madsen (lmadsen) 2011-01-04 13:54:53.000-0600 Agreed with pabelanger By: Lott Caskey (lottc) 2011-01-19 03:39:28.000-0600 Uploaded patch against trunk. Add reviewboard entry 1080 against asterisk-dev. https://reviewboard.asterisk.org/r/1080/ By: Leif Madsen (lmadsen) 2011-06-01 14:03:27 You'll have to re-license this code away from GPLv3 for it to be accepted. I think LGPL is fine, but I don't think GPLv2/3 is compatible with Asterisk's dual licensing. By: Lott Caskey (lottc) 2011-06-02 09:56:01 I would imagine my license agreement would supersede the in-file license reference. Especially indicated by the phrasing, "Digium may distribute and sublicense to others on any licensing terms..." But just to be clear, I have removed the in-file reference and submit this patch in accordance with the previously signed license agreement. Please let me know if this is not sufficient. Uploaded new patch against latest trunk. In this patch, a few new references to the AST_DATA_DIR/sounds have been replaced with AST_SOUND_DIR. | ||