[asterisk-dev] [Code Review] Add option for asterisk -rx timeout length

[Tilghman Lesher]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/573/#review1701
-----------------------------------------------------------


Actually, we could just get rid of the timeout altogether.  We've already added the "cli quit after" prefix, which forces an implicit 'quit' after the command finishes executing, anyway.

- Tilghman


On 2010-03-18 12:40:49, Russell Bryant wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/573/
> -----------------------------------------------------------
> 
> (Updated 2010-03-18 12:40:49)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> I noticed that when using asterisk -rx to execute "test execute all", I never get all of the output from the command.  The process exits early.  This patch at least helps that situation.  asterisk -rx is actually quite a hack.  This patch does not change the fact that it is a hack, but it allows you to control the timeout used before it exits.
> 
> Obviously, using astcli is a better solution, but this still seems like a worthwhile improvement to using asterisk -rx.
> 
> 
> This addresses bug 17049.
>     https://issues.asterisk.org/view.php?id=17049
> 
> 
> Diffs
> -----
> 
>   /trunk/Makefile 253259 
>   /trunk/doc/asterisk.8 253259 
>   /trunk/doc/asterisk.sgml 253259 
>   /trunk/main/asterisk.c 253259 
> 
> Diff: https://reviewboard.asterisk.org/r/573/diff
> 
> 
> Testing
> -------
> 
> With this patch, asterisk -rx "test execute all" will exit early.  asterisk -rx "test execute all" -w 10000 allows me to get all of the output (until someone writes a test that doesn't produce any output in 10 seconds ...).
> 
> 
> Thanks,
> 
> Russell
> 
>