On Fri, Mar 8, 2013 at 6:19 PM, Arran Cudbard-Bell
<a.cudbardb@freeradius.org> wrote:
On 8 Mar 2013, at 17:15, Avesh Agarwal <avesh.ncsu@gmail.com> wrote:
> Hello Alan,
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.
I agree with you. As the last conversation I had about it was with Alan, so I just addressed to him.
> 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:
In future please create patches against the master branch so we don't have to go through the pain of forward porting them.
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...
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.
But anyway thanks for doing this. I'll merge the patches manually, and make any required fixes.
I appreciate your help and effort you put in for this patch.
-Arran