[Home]

Summary:ASTERISK-26353: res_musiconhold: musiconhold seems to think that the general section is a class and issues warning
Reporter:Jonathan Harris (lardconcepts)Labels:
Date Opened:2016-09-10 08:39:23Date Closed:2017-03-09 17:01:58.000-0600
Priority:MinorRegression?Yes
Status:Closed/CompleteComponents:Resources/res_musiconhold
Versions:GIT Frequency of
Occurrence		Constant
Related
Issues:
Environment:Asterisk GIT-14-d7d22a2 built by root on a x86_64 running Linux on 2016-09-09 15:59:12 UTC Attachments:
Description:First report! Hope I'm doing it right - be gentle!

In the version of 14 I pulled from GIT and compiled yesterday, I'm seeing the following ERROR warning on the console:

[Sep 10 14:28:16] WARNING[4603]: res_musiconhold.c:1767 load_moh_classes: A directory must be specified for class 'general'!

This is the musiconhold.conf as provided by the samples - it's seems to be thinking that the general section is a class?!

[general]

[default]
mode=files
directory=moh

Comments:By: Asterisk Team (asteriskteam) 2016-09-10 08:39:23.941-0500

Thanks for creating a report! The issue has entered the triage process. That means the issue will wait in this status until a Bug Marshal has an opportunity to review the issue. Once the issue has been reviewed you will receive comments regarding the next steps towards resolution.

A good first step is for you to review the [Asterisk Issue Guidelines|https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines] if you haven't already. The guidelines detail what is expected from an Asterisk issue report.

Then, if you are submitting a patch, please review the [Patch Contribution Process|https://wiki.asterisk.org/wiki/display/AST/Patch+Contribution+Process].
By: Rusty Newton (rnewton) 2016-10-21 16:48:31.682-0500

Yeah that is goofy. I don't think there should be a general class, plus 'directory' isn't an option for the general section.

{noformat}
[Oct 21 16:40:38] VERBOSE[32057] loader.c: Loading res_musiconhold.so.
[Oct 21 16:40:38] DEBUG[32057] config.c: Parsing /etc/asterisk/musiconhold.conf
[Oct 21 16:40:38] VERBOSE[32057] config.c: Parsing '/etc/asterisk/musiconhold.conf': Found
[Oct 21 16:40:38] WARNING[32057] res_musiconhold.c: A directory must be specified for class 'general'!
[Oct 21 16:40:38] DEBUG[32057] res_musiconhold.c: Destroying MOH class 'general'
[Oct 21 16:40:38] DEBUG[32057] res_musiconhold.c: Scanning '/var/lib/asterisk/moh' for files for class 'default'
[Oct 21 16:40:38] VERBOSE[32057] pbx_app.c: Registered application 'MusicOnHold'
[Oct 21 16:40:38] VERBOSE[32057] pbx_app.c: Registered application 'StartMusicOnHold'
[Oct 21 16:40:38] VERBOSE[32057] pbx_app.c: Registered application 'StopMusicOnHold'
[Oct 21 16:40:38] VERBOSE[32057] loader.c: res_musiconhold.so => (Music On Hold Resource)
[Oct 21 16:40:38] DEBUG[32057] loader.c: START: res_musiconhold.so                             0 0
{noformat}
After adding directory=moh2 to the general section
{noformat}
[Oct 21 16:43:20] VERBOSE[32220] loader.c: Loading res_musiconhold.so.
[Oct 21 16:43:20] DEBUG[32220] config.c: Parsing /etc/asterisk/musiconhold.conf
[Oct 21 16:43:20] VERBOSE[32220] config.c: Parsing '/etc/asterisk/musiconhold.conf': Found
[Oct 21 16:43:20] WARNING[32220] res_musiconhold.c: Unknown option 'directory' in [general] section of musiconhold.conf
[Oct 21 16:43:20] WARNING[32220] res_musiconhold.c: A mode must be specified for class 'general'!
[Oct 21 16:43:20] DEBUG[32220] res_musiconhold.c: Destroying MOH class 'general'
[Oct 21 16:43:20] DEBUG[32220] res_musiconhold.c: Scanning '/var/lib/asterisk/moh' for files for class 'default'
[Oct 21 16:43:20] VERBOSE[32220] pbx_app.c: Registered application 'MusicOnHold'
[Oct 21 16:43:20] VERBOSE[32220] pbx_app.c: Registered application 'StartMusicOnHold'
[Oct 21 16:43:20] VERBOSE[32220] pbx_app.c: Registered application 'StopMusicOnHold'
[Oct 21 16:43:20] VERBOSE[32220] loader.c: res_musiconhold.so => (Music On Hold Resource)
[Oct 21 16:43:20] DEBUG[32220] loader.c: START: res_musiconhold.so                             0 0
{noformat}
By: Friendly Automation (friendly-automation) 2017-03-09 17:01:59.750-0600

Change 5145 merged by Joshua Colp:
res_musiconhold: moh general section is a class and issues warning

[https://gerrit.asterisk.org/5145|https://gerrit.asterisk.org/5145]
By: Friendly Automation (friendly-automation) 2017-03-09 18:33:11.507-0600

Change 5144 merged by zuul:
res_musiconhold: moh general section is a class and issues warning

[https://gerrit.asterisk.org/5144|https://gerrit.asterisk.org/5144]