Hi Greg <br><br>&gt;Balaji has been taking patches mailed to the list and storing them in<br>
&gt;his git repository (see HACKING and HACKING.pending), which I<br>
&gt;appreciate.  Patches sent to the list often don&#39;t have commit messages,<br>
&gt;though, and those are very important.  If the only issue is not being<br>
&gt;able to publish a git repository, then emailed patches with git<br>
&gt;format-patch are fine, because those end up with commit messages after<br>
&gt;git am.<br><br>Thanks. I have the quagga-mbgp branch in the git hub and the code is also available in <a href="http://quagga.net">quagga.net</a>. I have created an other repo called quagga-devel in git hub which contains a clone of quagga-next . Is it possible for you to take in those changes as such ? Paul had done a round of review of that. and i am making changes in the CLI too to make it Cisco like as there are lot of repetitions in the CLI. I would also work on other features and fixes and push them into quagga-devel soon. <br>
<br>Let me know how do you wanna proceed on this.<br><br>Cheers,<br>  - Balaji<br><br>
<br><br><div class="gmail_quote">On Tue, Nov 30, 2010 at 6:41 PM, Greg Troxel <span dir="ltr">&lt;<a href="mailto:gdt@ir.bbn.com">gdt@ir.bbn.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
@balaji: Thanks for explaining this and keeping patches organized.<br>
<br>
@quagga-dev:<br>
<br>
I have indeed been trying to apply some patches.  The basic difficulty<br>
is that I feel the existing codebase is fragile and underdocumented in<br>
terms of calling conventions, invariants, etc.  Reviewing each patch to<br>
be sure that it won&#39;t cause regressions is very tricky, and I&#39;ve found<br>
that I&#39;m not comfortable yet with a significant number of them.<br>
<br>
I did put one on a branch and asked for regression testing, but haven&#39;t<br>
yet heard any reports.  It&#39;s on feature.bgp-vty-view-af.  If a few<br>
people running BGP for real told me that they updated to that and it was<br>
fine, I&#39;d merge it master.<br>
<br>
I will be trying to examine patches and for each either apply it,<br>
feature branch it and call for testing, or explain on quagga-dev why I<br>
didn&#39;t do one of those.<br>
<br>
It&#39;s easiest for me if proposed changes are in a publically-accessible<br>
git repository that&#39;s essentially a clone of the official repository,<br>
with each logically-related group of changes on a feature branch off<br>
master.  That means changes can be merged independently.<br>
<br>
Balaji has been taking patches mailed to the list and storing them in<br>
his git repository (see HACKING and HACKING.pending), which I<br>
appreciate.  Patches sent to the list often don&#39;t have commit messages,<br>
though, and those are very important.  If the only issue is not being<br>
able to publish a git repository, then emailed patches with git<br>
format-patch are fine, because those end up with commit messages after<br>
git am.<br>
<br>
</blockquote></div><br>