<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/1917/">https://reviewboard.asterisk.org/r/1917/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On May 16th, 2012, 3:09 p.m., <b>Paul Belanger</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">What sort of upgrade path are we expecting between chan_jingle and chan_jingle2? I didn&#39;t see anything specific on the wiki pages. If I remember right, this is not a drop in replacement so it might be good to have it documented on the wiki what is and what is not.

Additionally, I mentioned this in passing, while it is cool we have version 2 of jingle, I&#39;m not a fan of appending 2 to the channel name.  Perhaps something like chan_xmpp_jingle? At first it is kinda ugly, but seems to grow on me as I look at it more.</pre>
 </blockquote>







</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I also dislike adding &#39;2&#39; to the end. I&#39;m not really picky, and what Paul has suggested seems reasonable.</pre>
<br />








<p>- Leif</p>


<br />
<p>On May 13th, 2012, 12:15 p.m., Joshua Colp wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/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 Joshua Colp.</div>


<p style="color: grey;"><i>Updated May 13, 2012, 12:15 p.m.</i></p>




<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 is a new channel driver written from scratch for the Jingle, Google Jingle, and Google Talk protocols. It has been written to the specs available and tested extensively.

ICE and STUN support for Jingle uses the new ICE/STUN/TURN support which is present in another review. (Please do not review any of that code in this review)
STUN support for Google uses the existing STUN implementation, as the new support is not compatible with it.</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;">Tested audio calls with following:

GMail Google Talk Plug-in (and video)
Google Voice
Jitsi (and video)
Psi
OneTeam

* Included varying codecs (ulaw, speex, g722, etc)

Tested ringing, hold, and unhold with following:

Jitsi

Other clients do not support this.</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>/trunk/channels/chan_jingle2.c <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/channels/chan_sip.c <span style="color: grey">(365451)</span></li>

 <li>/trunk/configs/jingle2.conf.sample <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/configs/rtp.conf.sample <span style="color: grey">(365451)</span></li>

 <li>/trunk/include/asterisk/jabber.h <span style="color: grey">(365451)</span></li>

 <li>/trunk/include/asterisk/jingle.h <span style="color: grey">(365451)</span></li>

 <li>/trunk/include/asterisk/rtp_engine.h <span style="color: grey">(365451)</span></li>

 <li>/trunk/main/rtp_engine.c <span style="color: grey">(365451)</span></li>

 <li>/trunk/res/Makefile <span style="color: grey">(365451)</span></li>

 <li>/trunk/res/res_jabber.c <span style="color: grey">(365451)</span></li>

 <li>/trunk/res/res_rtp_asterisk.c <span style="color: grey">(365451)</span></li>

</ul>

<p><a href="https://reviewboard.asterisk.org/r/1917/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>