[asterisk-bugs] [Asterisk 0016906]: [patch] moh files install under datadir, at runtime: under varlibdir

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16906 
====================================================================== 
Reported By:                tzafrir
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   16906
Category:                   Resources/res_musiconhold
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     acknowledged
Asterisk Version:           1.6.2.3-rc2 
JIRA:                       SWP-980 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-02-25 15:45 CST
Last Modified:              2010-10-16 05:03 CDT
====================================================================== 
Summary:                    [patch] moh files install under datadir, at runtime:
under varlibdir
Description: 
In sounds/Makefile , the moh files are installed to:

  MOH_DIR:=$(DESTDIR)$(ASTDATADIR)/moh

However in res/res_musiconhold.c , in moh_scan_files we have (1.6.2) -

        if (class->dir[0] != '/') {
                ast_copy_string(dir_path, ast_config_AST_VAR_DIR,
sizeof(dir_path));
                strncat(dir_path, "/", sizeof(dir_path) - 1);
                strncat(dir_path, class->dir, sizeof(dir_path) - 1);


Thus if astdatadir != astvarrundir , res_musiconh will fail to find the
files with the default configuration.
====================================================================== 

---------------------------------------------------------------------- 
 (0128118) svnbot (reporter) - 2010-10-16 05:03
 https://issues.asterisk.org/view.php?id=16906#c128118 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 292049

U   branches/1.6.2/configs/musiconhold.conf.sample
U   branches/1.6.2/res/res_musiconhold.c

------------------------------------------------------------------------
r292049 | tzafrir | 2010-10-16 05:03:05 -0500 (Sat, 16 Oct 2010) | 15
lines

Base directory for MOH should be ASTDATADIR

If the directive 'directory' is relative, make it relative to the
datadir, rather than to the varlibdir. In the sample configuration
it is relative ('moh').

This has no effect unless you have actively set the datadir explicitly
(at build time or at run time).

(closes issue https://issues.asterisk.org/view.php?id=16906)
Patches:
      moh_datadir uploaded by tzafrir (license 46)

Review: https://reviewboard.asterisk.org/r/974/

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=292049 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-10-16 05:03 svnbot         Checkin                                      
2010-10-16 05:03 svnbot         Note Added: 0128118                          
======================================================================