<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/2708/">https://reviewboard.asterisk.org/r/2708/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">A few things to add:
* In the CHANGES and UPGRADE file, note that libxslt is now optionally used to improve XML documentation. If libxslt isn't present, some XML documentation may be incomplete (it will be)
* In the make_deps script, add libxslt
* If they don't have libxslt, use a #else and emit a NOTICE at least that XSLT support wasn't found. XML documentation may be incomplete (it will be)
That really is the only issue I have with this: if you don't have libxslt, your event documentation is going to be lacking. As long as it doesn't blow up, however, that isn't the worst thing in the world (and AMI event documentation is less useful in CLI than in other mediums)
</pre>
<br />
<p>- Matt</p>
<br />
<p>On July 26th, 2013, 8:46 p.m. UTC, opticron wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By opticron.</div>
<p style="color: grey;"><i>Updated July 26, 2013, 8:46 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This adds xslt handling support to Asterisk and a stylesheet that generates channel and bridge snapshots on the fly with appropriate prefixes, all from tags like: <channel_snapshot prefix="Dest"/>
This also corrects existing documentation to use this new single-tag solution.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Verified that XML is still parsed properly by Asterisk when libxslt is present and fails to expand the new tags when it isn't (without failing catastrophically).</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/configure <span style="color: grey">(UNKNOWN)</span></li>
<li>/trunk/Makefile <span style="color: grey">(395557)</span></li>
<li>/trunk/apps/app_agent_pool.c <span style="color: grey">(395557)</span></li>
<li>/trunk/apps/app_meetme.c <span style="color: grey">(395557)</span></li>
<li>/trunk/apps/app_minivm.c <span style="color: grey">(395557)</span></li>
<li>/trunk/apps/app_queue.c <span style="color: grey">(395557)</span></li>
<li>/trunk/apps/app_stack.c <span style="color: grey">(395557)</span></li>
<li>/trunk/apps/confbridge/confbridge_manager.c <span style="color: grey">(395557)</span></li>
<li>/trunk/channels/chan_dahdi.c <span style="color: grey">(395557)</span></li>
<li>/trunk/channels/chan_sip.c <span style="color: grey">(395557)</span></li>
<li>/trunk/channels/sig_pri.c <span style="color: grey">(395557)</span></li>
<li>/trunk/configure.ac <span style="color: grey">(395557)</span></li>
<li>/trunk/doc/appdocsxml.dtd <span style="color: grey">(395557)</span></li>
<li>/trunk/doc/snapshots.xslt <span style="color: grey">(PRE-CREATION)</span></li>
<li>/trunk/funcs/func_global.c <span style="color: grey">(395557)</span></li>
<li>/trunk/include/asterisk/autoconfig.h.in <span style="color: grey">(395557)</span></li>
<li>/trunk/main/Makefile <span style="color: grey">(395557)</span></li>
<li>/trunk/main/aoc.c <span style="color: grey">(395557)</span></li>
<li>/trunk/main/core_local.c <span style="color: grey">(395557)</span></li>
<li>/trunk/main/features.c <span style="color: grey">(395557)</span></li>
<li>/trunk/main/manager.c <span style="color: grey">(395557)</span></li>
<li>/trunk/main/manager_bridges.c <span style="color: grey">(395557)</span></li>
<li>/trunk/main/manager_channels.c <span style="color: grey">(395557)</span></li>
<li>/trunk/main/manager_mwi.c <span style="color: grey">(395557)</span></li>
<li>/trunk/main/rtp_engine.c <span style="color: grey">(395557)</span></li>
<li>/trunk/main/stasis_bridges.c <span style="color: grey">(395557)</span></li>
<li>/trunk/main/stasis_channels.c <span style="color: grey">(395557)</span></li>
<li>/trunk/main/xml.c <span style="color: grey">(395557)</span></li>
<li>/trunk/makeopts.in <span style="color: grey">(395557)</span></li>
<li>/trunk/res/parking/parking_manager.c <span style="color: grey">(395557)</span></li>
<li>/trunk/res/res_agi.c <span style="color: grey">(395557)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2708/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>