[Home]

Summary:ASTERISK-15789: [patch] trivial patch to notifiy CLI when module loaded / unloaded.
Reporter:Paul Belanger (pabelanger)Labels:
Date Opened:2010-05-07 10:28:29Date Closed:2010-05-12 15:03:58
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) cli.modules.patch
( 1) Unloaded.module.patch
Description:Currently, there is no way to notify the CLI the result of load/unload modules.

asterisk -rx "module load app_dial.so"
asterisk -rx 'module unload app_dial.so"

****** ADDITIONAL INFORMATION ******

This patch adds the functionality;

 asterisk -rx "module load app_dial.so"
 Loaded app_dial.so
 asterisk -rx "module unload app_dial.so"
 Unloaded app_dial.so

Also now generates CLI error when loading unknown module:

 asterisk -rx "module load app_bobsmith.so"
 Unable to load module app_bobsmith.so
 Command 'module load app_bobsmith.so' failed.
Comments:By: Paul Belanger (pabelanger) 2010-05-07 11:24:49

v2: Actually output to CLI when loading/unloading modules.

By: Russell Bryant (russell) 2010-05-11 10:22:22

Looks fine to me, ship it

By: Digium Subversion (svnbot) 2010-05-12 14:59:17

Repository: asterisk
Revision: 262800

U   trunk/main/cli.c
U   trunk/main/loader.c

------------------------------------------------------------------------
r262800 | pabelanger | 2010-05-12 14:59:16 -0500 (Wed, 12 May 2010) | 8 lines

Notify CLI when modules is loaded / unloaded

(closes issue ASTERISK-15789)
Reported by: pabelanger
Patches:
     cli.modules.patch uploaded by pabelanger (license 224)
Tested by: pabelanger, russell

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

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

By: Digium Subversion (svnbot) 2010-05-12 15:01:40

Repository: asterisk
Revision: 262801

_U  branches/1.6.2/
U   branches/1.6.2/main/cli.c
U   branches/1.6.2/main/loader.c

------------------------------------------------------------------------
r262801 | pabelanger | 2010-05-12 15:01:39 -0500 (Wed, 12 May 2010) | 15 lines

Merged revisions 262800 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r262800 | pabelanger | 2010-05-12 15:59:16 -0400 (Wed, 12 May 2010) | 8 lines
 
 Notify CLI when modules is loaded / unloaded
 
 (closes issue ASTERISK-15789)
 Reported by: pabelanger
 Patches:
       cli.modules.patch uploaded by pabelanger (license 224)
 Tested by: pabelanger, russell
........

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

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

By: Digium Subversion (svnbot) 2010-05-12 15:02:59

Repository: asterisk
Revision: 262802

_U  branches/1.6.1/
U   branches/1.6.1/main/cli.c
U   branches/1.6.1/main/loader.c

------------------------------------------------------------------------
r262802 | pabelanger | 2010-05-12 15:02:58 -0500 (Wed, 12 May 2010) | 15 lines

Merged revisions 262800 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r262800 | pabelanger | 2010-05-12 15:59:16 -0400 (Wed, 12 May 2010) | 8 lines
 
 Notify CLI when modules is loaded / unloaded
 
 (closes issue ASTERISK-15789)
 Reported by: pabelanger
 Patches:
       cli.modules.patch uploaded by pabelanger (license 224)
 Tested by: pabelanger, russell
........

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

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

By: Digium Subversion (svnbot) 2010-05-12 15:03:58

Repository: asterisk
Revision: 262803

_U  branches/1.6.0/
U   branches/1.6.0/main/cli.c
U   branches/1.6.0/main/loader.c

------------------------------------------------------------------------
r262803 | pabelanger | 2010-05-12 15:03:58 -0500 (Wed, 12 May 2010) | 15 lines

Merged revisions 262800 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r262800 | pabelanger | 2010-05-12 15:59:16 -0400 (Wed, 12 May 2010) | 8 lines
 
 Notify CLI when modules is loaded / unloaded
 
 (closes issue ASTERISK-15789)
 Reported by: pabelanger
 Patches:
       cli.modules.patch uploaded by pabelanger (license 224)
 Tested by: pabelanger, russell
........

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

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