<div dir="ltr"><div>i think you have not understood the sense of this contribute. I don't have experience with git, so i thought that create a my own repository was a better idea.</div><div>I don't known atlassian or jira, and i'm using atlassian instead of github only because it's free and i can create private repository.</div>
<div><br></div><div>It is obvious that </div><div>//MM</div><div>...</div><div>code</div><div>...</div><div>//ENDMM</div><div>cannot be pushed in the main repository of freeradius. I'm using this only for track changes to code. This is a bad idea, ok, but, as i mentioned before, it's due to my inexperience with git. </div>
<div>That i can do is: </div><div>- create a new public repository (on bitbucket or github, it's the same in my case!)</div><div>- init this repo with the original code of the rlm_counter module</div><div>- Pushing new code gradually</div>
<div><br></div><div>Sincerely, i don't know how create a fork on github and i don't know if it is useful for a single module composed of a single file.</div><div>This is not my primary job, i wanted write this code because it is a need in my company and i thought that there may be other users with same needs.</div>
<div>I'm sorry if i done something stupid.</div><div><br></div><div>MM</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-03-12 0:01 GMT+01:00 Alan DeKok <span dir="ltr"><<a href="mailto:aland@deployingradius.com" target="_blank">aland@deployingradius.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">Marco Marino wrote:<br>
> It's a modified version of rlm_counter.<br>
<br>
</div>  It's easier to see what's changed if you start off with the main<br>
FreeRADIUS git repository.  There is no benefit to pulling out<br>
rlm_counter, and putting it into it's own git repository.<br>
<br>
  Doing that makes it impossible to tell what's changed.  It's<br>
impossible to use git to pull the changes back into the main repository.<br>
 Instead of using git, we've got to apply the patches by hand.<br>
<br>
  i.e. this negates *all* benefit of using git.<br>
<br>
  Please use github to fork the main FreeRADIUS repository.  Then,<br>
commit your patches one by one.  Then, submit a "pull" request.<br>
<div class=""><br>
> In the repository there is a file named counter that contains an<br>
> example for a configuration file. Use this in mods-available/counter<br>
> and create a symbolic link in mods-enabled/<br>
<br>
</div>  As opposed to just modifying the raddb/mods-available/counter file<br>
directly, and tracking those changes in git.<br>
<br>
  You've made it as difficult as possible for us to accept these<br>
changes.  Why?<br>
<br>
  And things like this in the code will NEVER be accepted:<br>
<br>
//MM<br>
/*<br>
*       Define a structure for single counter object<br>
*/<br>
typedef struct single_counter_t {<br>
        char *name;<br>
        int check;<br>
        int reply;<br>
        struct single_counter_t *next;<br>
} single_counter_t;<br>
//ENDMM<br>
<br>
  There is NO REASON to put your initials into the code.  If you need to<br>
track who changed the code, use "git".<br>
<div class="HOEnZb"><div class="h5"><br>
  Alan DeKok.<br>
-<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></div>