[asterisk-bugs] [Asterisk 0013153]: [patch] set ASTVARRUNDIR=$(localstatedir)/run/asterisk as default path

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13153 
====================================================================== 
Reported By:                pabelanger
Assigned To:                mvanbaak
====================================================================== 
Project:                    Asterisk
Issue ID:                   13153
Category:                   Core/BuildSystem
Reproducibility:            N/A
Severity:                   feature
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.0 
SVN Revision (number only!): 133524 
Request Review:              
====================================================================== 
Date Submitted:             2008-07-24 17:14 CDT
Last Modified:              2009-02-21 07:13 CST
====================================================================== 
Summary:                    [patch] set
ASTVARRUNDIR=$(localstatedir)/run/asterisk as default path
Description: 
A simple patch to change the default location for ASTVARRUNDIR.

The reason for this revolves around running asterisk as a non-root user. 
With this patch, we can setup specific permission on the folder for our
non-root user to control.

Without this patch, there is no way to control the permission on the
ASTVARRUNDIR path because other application also write to this path.

A conversation on #asterisk-dev with seanbright raised no major
objections.
====================================================================== 

---------------------------------------------------------------------- 
 (0100497) svnbot (reporter) - 2009-02-21 07:13
 http://bugs.digium.com/view.php?id=13153#c100497 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 177852

U   trunk/Makefile
U   trunk/contrib/init.d/rc.archlinux.asterisk
U   trunk/contrib/init.d/rc.debian.asterisk
U   trunk/contrib/scripts/safe_asterisk

------------------------------------------------------------------------
r177852 | mvanbaak | 2009-02-21 07:13:36 -0600 (Sat, 21 Feb 2009) | 18
lines

set ASTVARRUNDIR=$(localstatedir)/run/asterisk as default path

When running asterisk as non-root and without this patch the pidfile wants
to go into /var/run/asterisk.pid. This directory is not writable for
the non-root user and changing permissions is not an option.
Putting it in /var/run/asterisk/asterisk.pid makes it possible
to set permissions on the /var/run/asterisk dir so everything
works as it should be.

Patched committed is based on pabelanger's patch.

(closes issue http://bugs.digium.com/view.php?id=13153)
Reported by: pabelanger
Patches:
      2009012900_bug13153-nonrootscripts.diff.txt uploaded by mvanbaak
(license 7)

Review: http://reviewboard.digium.com/r/139/

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-02-21 07:13 svnbot         Checkin                                      
2009-02-21 07:13 svnbot         Note Added: 0100497                          
======================================================================