<font face="arial,helvetica,sans-serif">Hi Denis,<br></font><br><div class="gmail_quote">On Fri, Feb 10, 2012 at 6:57 AM, Denis Ovsienko <span dir="ltr">&lt;<a href="mailto:infrastation@yandex.ru">infrastation@yandex.ru</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">01.02.2012, 13:52, &quot;Subbaiah Venkata&quot; &lt;<a href="mailto:svenkata@google.com">svenkata@google.com</a>&gt;:<br>

<div class="im">&gt; Hi All,<br>
&gt;<br>
&gt; New ISIS branch is pushed to google public repository.<br>
&gt; <a href="http://code.google.com/p/google-quagga/source/browse/?name=is-is" target="_blank">http://code.google.com/p/google-quagga/source/browse/?name=is-is</a><br>
&gt;<br>
</div>&gt; This branch has support for new features and bug fixes.Please let me know if you have any questions.<br>
<br>
Hello, Subbaiah.<br>
<br>
Thank you for publishing this work. This is the second publication in the name of Google, and the final goal, as far as I take it, is to converge the branches in foreseeable future. It will require some work on the project&#39;s side as well.<br>
</blockquote><div><br></div><div>Agree, converging the branches is good for all of us in the community.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
I have briefly studied the contents of commits b1b73b9 and f17b498. First of all, proofreading is hard, because changes are not isolated from each other, the change to IS-IS, BGP, library functions, zserv and indentation are mixed together. Just in case the changes were originally made and verified in some ordered way and later joined in a single commit for the sole purpose of publication, discussing the original isolated changes one at a time would help making through the merge. Is it possible?<br>
</blockquote><div><br></div><div>Quagga ISIS was not in a production ready state. I have touched almost all parts of ISIS code and it take lot of time to cherry pick and get them reviewed one at a time. Frankly there are 100s of changes. I think a protocol code should go into slow moving small piece meal change reviews when it gets into a maintenance mode, which ISIS is not in that state yet. So I suggest we merge the code, test and commit to the head.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Then, there is a series of IS-IS commits, which noone was able to verify so far:<br>
<br>
d83bf5a<br>
30a7558<br>
dfafeae<br>
b775c38<br>
9632e3e<br>
<br>
The commit IDs belong to quagga-RE repo only. Do you have the time and will to review these? They seem to be very closely related.<br></blockquote><div><br></div><div>Yes, I have some time to review Quagga changes.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
--<br>
 Denis Ovsienko<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Quagga-dev mailing list<br>
<a href="mailto:Quagga-dev@lists.quagga.net">Quagga-dev@lists.quagga.net</a><br>
<a href="http://lists.quagga.net/mailman/listinfo/quagga-dev" target="_blank">http://lists.quagga.net/mailman/listinfo/quagga-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>--<div>Subbaiah Venkata</div><div>Manager, Google Inc</div><div>1600 Amphetheatre Parkway</div><div>Mountain View, CA 94043</div><br>