[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:                tzafrir
====================================================================== 
Project:                    Asterisk
Issue ID:                   16906
Category:                   Resources/res_musiconhold
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
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:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-02-25 15:45 CST
Last Modified:              2010-10-16 05:47 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.
====================================================================== 

---------------------------------------------------------------------- 
 (0128119) svnbot (reporter) - 2010-10-16 05:47
 https://issues.asterisk.org/view.php?id=16906#c128119 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 292050

_U  branches/1.8/
U   branches/1.8/configs/musiconhold.conf.sample
U   branches/1.8/res/res_musiconhold.c

------------------------------------------------------------------------
r292050 | tzafrir | 2010-10-16 05:47:01 -0500 (Sat, 16 Oct 2010) | 22
lines

Merged revisions 292049 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

........
  r292049 | tzafrir | 2010-10-16 12:03:04 +0200 (?\215?\169', 16
?\215?\144?\215?\149?\215?\167 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=292050 

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