[Home]

Summary:ASTERISK-05175: [patch] Asterisk checks for remote connection as 'rasterisk' but does not install rasterisk binary
Reporter:Tilghman Lesher (tilghman)Labels:
Date Opened:2005-09-28 11:57:49Date Closed:2008-01-15 15:49:40.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20050928__rasterisk__2.diff.txt
Description:Currently, if Asterisk is run as the binary 'rasterisk', it assumes remote connection.  However, the 'rasterisk' binary is never installed.

Also, forces 'rasterisk' name if the remote option is turned on.  This makes it easier for init scripts to tell if asterisk has truly shutdown yet during a restart or shutdown sequence.
Comments:By: Olle Johansson (oej) 2005-09-28 13:50:02

The for loop does not follow the coding guidelines.

Sorry, Corydon, couldn't resist ;-)
By: Tilghman Lesher (tilghman) 2005-09-28 14:05:37

Fair enough.
By: Brian West (bkw918) 2005-09-28 21:06:25

It should only be a symlink shouldn't it?

/b
By: Brian West (bkw918) 2005-09-28 21:11:05

Yes a symlink is all thats needed.

/b
By: Brian West (bkw918) 2005-09-28 21:11:52

But shouldn't the default be just to do -r if you don't tell it and asterisk is already running it can already tell you its already running and to use -r wouldn't just be more user friendly to just assume -r in those cases?

/b
By: Kevin P. Fleming (kpfleming) 2005-09-28 22:42:08

Committed to CVS HEAD, with mod to use symlink instead of copying binary. Thanks!
By: Digium Subversion (svnbot) 2008-01-15 15:49:40.000-0600

Repository: asterisk
Revision: 6688

U   trunk/Makefile
U   trunk/asterisk.c

------------------------------------------------------------------------
r6688 | kpfleming | 2008-01-15 15:49:39 -0600 (Tue, 15 Jan 2008) | 2 lines

install 'rasterisk' symlink and update process name when used remotely (issue ASTERISK-5175 with minor mod)

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

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