<br><br><div class="gmail_quote">On Fri, Mar 8, 2013 at 6:19 PM, Arran Cudbard-Bell <span dir="ltr"><<a href="mailto:a.cudbardb@freeradius.org" target="_blank">a.cudbardb@freeradius.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On 8 Mar 2013, at 17:15, Avesh Agarwal <<a href="mailto:avesh.ncsu@gmail.com">avesh.ncsu@gmail.com</a>> wrote:<br>
<br>
> Hello Alan,<br>
<br>
Although Alan is the project leader, many other people contribute to FreeRADIUS. Please address messages to the wider list if you're not emailing a developer directly.<br>
<div class="im"><br></div></blockquote><div>I agree with you. As the last conversation I had about it was with Alan, so I just addressed to him.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">
> I have created a forked version of latest freeradius v2 branch on github and merged the updated EAP-TNC support from tnch@fhh. The link is here:<br>
<br>
</div>In future please create patches against the master branch so we don't have to go through the pain of forward porting them.<br>
<br>
You went to the trouble of forking FreeRADIUS on GitHub, creating the commit, but inexplicably decided not to open a pull request and instead posted the commit to the developers list? Why? It makes it harder to merge, and means your code doesn't get submitted to Travis CI...<br>

<br></blockquote><div>I agree with you here too. As it was first time patch from me for freeradius, so I just thought to show the patch to the developers here before opening a pull request. Thanks for letting me know about and I will keep it in mind next time. <br>
 <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
But anyway thanks for doing this. I'll merge the patches manually, and make any required fixes.<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div>I appreciate your help and effort you put in for this patch. <br></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">
-Arran<br>
</font></span><div class="HOEnZb"><div class="h5">-<br>
List info/subscribe/unsubscribe? See <a href="http://www.freeradius.org/list/devel.html" target="_blank">http://www.freeradius.org/list/devel.html</a><br>
</div></div></blockquote></div><br>