[asterisk-dev] Janitor Project: NEW_CLI conversion

Steve Totaro stotaro at first-notification.com
Sun Sep 16 18:55:48 CDT 2007


Ah, so I am laughing at myself then, wouldn't be the first nor the 
last.   :-)

Thanks,
Steve

Jason Parker wrote:
> Sean opened that bug report, and had a patch to go along with it.  He was telling other developers that if they would like to come up with patches, that bug would be the most appropriate place for them. :)
>
> ----- "Steve Totaro" <stotaro at first-notification.com> wrote:
>   
>> It just struck me as funny that you were telling an employee of Digium
>>
>> who works as a Software Engineer that he could post patches on Mantis.
>>  
>> I figured you must be joking but apparently not which makes it even
>> funnier.
>>
>> Maybe it's just me but it made me laugh.
>>
>> Thanks,
>> Steve
>>
>> Sean Bright wrote:
>>     
>>> Huh?
>>>
>>> On 9/15/07, *Steve Totaro* <stotaro at first-notification.com 
>>> <mailto:stotaro at first-notification.com>> wrote:
>>>
>>>     LOL
>>>
>>>     Sean Bright wrote:
>>>     > You can post patches here in Mantis:
>>>     >
>>>     >     http://bugs.digium.com/view.php?id=10724
>>>     >
>>>     > Sean
>>>     >
>>>     > On 9/14/07, Russell Bryant <russell at digium.com
>>>     <mailto:russell at digium.com>> wrote:
>>>     >
>>>     >> It has been a while since I posted a "janitor project" to
>>>       
>> this
>>     
>>>     list, so here is one.
>>>     >>
>>>     >> In trunk, we have a new method for defining CLI commands
>>>       
>> that
>>     
>>>     only requires
>>>     >> creating one function, has more features, and is less prone
>>>       
>> to
>>     
>>>     errors.  Most of
>>>     >> the code for implementing CLI commands still uses the old
>>>     method.  However, it
>>>     >> would be a welcome change to convert any CLI commands to the
>>>     new style.
>>>     >>
>>>     >> Examples of using the new style can be found in main/cli.c
>>>       
>> and
>>     
>>>     >> channels/chan_oss.c, among other places.  Search for
>>>       
>> "NEW_CLI"
>>     
>>>     and you will see
>>>     >> where CLI commands have been defined in the new style.
>>>     >>
>>>     >> If you have any questions, let me know!
>>>     >>
>>>     >> --
>>>     >> Russell Bryant
>>>     >> Software Engineer
>>>     >> Digium, Inc.
>>>     >>
>
>
>   




More information about the asterisk-dev mailing list