[asterisk-dev] Deprecate every '* reload' CLI command?

[Tomás Laureano Peralta Tormey]

I agree with the idea of starting a new Janitor to update the modules to use
'<module> reload' and change the behaviour of module reload to unload and
load the module. This way, '<module> reload' should reparse the
configuration file and 'module reload' should unload and load the module.
Also, I think this should be documented in some doc for the sake of
consistency to users and developers in future releases.

Best regards, Tomás.

On Nov 27, 2007 1:39 PM, Eliel Sardanons <eliel at eliel.com.ar> wrote:

> We could start a janitor for creating a 'foo reload' and we could make
> de 'module reload *.so' do a module unload; module load
>
>
> On 11/27/07, Russell Bryant <russell at digium.com> wrote:
> > Kevin P. Fleming wrote:
> > > I would be surprised if any module's 'reload' command actually did
> much
> > > of anything different than 'module reload' for the same module does,
> but
> > > as BJ points out there certainly is no reason that they have to
> provide
> > > the same functionality.
> >
> > They better not do anything different.  :)  If they do, I would
> definitely
> > consider it a bug.  That would be frustratingly inconsistent.
> >
> > > I think adding a 'voicemail reload' command is a fine idea.
> >
> > Well, I guess I agree that it is fine because it is a bit more user
> friendly
> > than "module reload app_voicemail.so".  However, I don't think we can
> leave the
> > conversation at that.
> >
> > The Asterisk CLI syntax got significantly switched around for Asterisk
> 1.4 for
> > the sake of consistency.  This issue points to another area where the
> Asterisk
> > CLI is not consistent.  We need to pick one way or the other and stick
> with it.
> >  If people really like the idea of having individual reload CLI commands
> like
> > "voicemail reload", then I say that we should:
> >
> > 1) Start a janitor project to go through the code base and implement
> "foo
> > reload" CLI commands for _every_ module that implements a reload module
> callback
> > function.
> >
> > 2) Mark the "module reload res_foo.so" syntax as deprecated and
> eventually
> > completely remove it.  This syntax was always confusing to me when I
> started
> > with Asterisk, because I thought it was equivalent to "module unload /
> module load".
> >
> > --
> > Russell Bryant
> > Senior Software Engineer
> > Open Source Team Lead
> > Digium, Inc.
> >
> > _______________________________________________
> > --Bandwidth and Colocation Provided by http://www.api-digital.com--
> >
> > asterisk-dev mailing list
> > To UNSUBSCRIBE or update options visit:
> >    http://lists.digium.com/mailman/listinfo/asterisk-dev
> >
>
>
> --
> Eliel Sardañons
>
> _______________________________________________
> --Bandwidth and Colocation Provided by http://www.api-digital.com--
>
> asterisk-dev mailing list
> To UNSUBSCRIBE or update options visit:
>   http://lists.digium.com/mailman/listinfo/asterisk-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.digium.com/pipermail/asterisk-dev/attachments/20071127/ee843d40/attachment-0001.htm