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

(Updated June 26, 2014, 2:50 p.m.)


Review request for Asterisk Developers, Matt Jordan and rmudgett.


Changes
-------

Address Richard's feedback


Repository: Asterisk


Description
-------

A series of AMI commands, these provide equivalents to the following CLI 
commands:
'pri set debug <level> span <span>' via PRISetDebug
'pri set debug file <output-file>'  via PRISetDebugFile

plus a bit of an odd one out with PRIUnsetDebugFile... doesn't really have a 
CLI equivalent, but it allows file logging for PRI to be disabled. This can 
also be accomplished by using 'pri set debug off span <span>', but I believe 
that's undesired behavior (because that should be dealing with per-span 
debugging and just completely ignores that other spans might be using file 
output) and have filed a bug report for it here: 
https://issues.asterisk.org/jira/browse/ASTERISK-23938


Diffs (updated)
-----

  /trunk/channels/chan_dahdi.c 417163 
  /trunk/CHANGES 417163 

Diff: https://reviewboard.asterisk.org/r/3681/diff/


Testing
-------

Used each AMI command with and without all parameters to make sure all error 
messages could be observed as expected and that the commands did what I 
expected them to do.
Reviewed documentation for AMI commands within Asterisk


Thanks,

Jonathan Rose

-- 
_____________________________________________________________________
-- 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

Reply via email to