[asterisk-commits] mvanbaak: trunk r208813 - in /trunk: configs/cli_aliases.conf.sample main/cli.c

[SVN commits to the Asterisk project]

Author: mvanbaak
Date: Sat Jul 25 07:03:25 2009
New Revision: 208813

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=208813
Log:
add default alias reload to run module reload.

Requiring 'module reload' to reload everything, including
core etc makes russell very unhappy.

The default configuration already loads the 'friendly' aliases template.
Added 'reload=module reload' to that template.

Also removed the comment in main/cli.c that reload should come back.

Modified:
    trunk/configs/cli_aliases.conf.sample
    trunk/main/cli.c

Modified: trunk/configs/cli_aliases.conf.sample
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/configs/cli_aliases.conf.sample?view=diff&rev=208813&r1=208812&r2=208813
==============================================================================
--- trunk/configs/cli_aliases.conf.sample (original)
+++ trunk/configs/cli_aliases.conf.sample Sat Jul 25 07:03:25 2009
@@ -27,6 +27,7 @@
 originate=channel originate
 help=core show help
 pri intense debug span=pri set debug 2 span
+reload=module reload
 
 ; CLI Alias Templates
 ; -------------------

Modified: trunk/main/cli.c
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/main/cli.c?view=diff&rev=208813&r1=208812&r2=208813
==============================================================================
--- trunk/main/cli.c (original)
+++ trunk/main/cli.c Sat Jul 25 07:03:25 2009
@@ -266,11 +266,6 @@
 	return CLI_SUCCESS;
 }
 
-/*!
- * \bug The command "reload" needs to be re-added.  Requiring "module reload" to reload everything,
- *      including core stuff, doesn't really make sense.  Adding a "core reload" to reload everything,
- *      including modules, also doesn't really make sense.  So, we should put "reload" back.
- */
 static char *handle_reload(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)
 {
 	int x;