<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 17, 2014 at 3:40 PM, Jérémy Lal <span dir="ltr"><<a href="mailto:kapouer@melix.org" target="_blank">kapouer@melix.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le vendredi 17 octobre 2014 à 15:04 -0700, Andrew Kelley a écrit :<br>
<div><div class="h5">> On Fri, Oct 17, 2014 at 12:15 AM, Jérémy Lal <<a href="mailto:kapouer@melix.org">kapouer@melix.org</a>> wrote:<br>
><br>
> > Le jeudi 16 octobre 2014 à 14:17 -0700, Andrew Kelley a écrit :<br>
> > > On Wed, Oct 15, 2014 at 12:44 AM, Jérémy Lal <<a href="mailto:kapouer@melix.org">kapouer@melix.org</a>> wrote:<br>
> > ><br>
> > > > > What should I do?<br>
> > > ><br>
> > > > Since both have same API, i'd add your patches as quilt patches to<br>
> > > > node-findit that is already in debian and add a binary package<br>
> > > > node-findit2 (symlinking /usr/lib/nodejs/findit to findit2).<br>
> > > ><br>
> > > > Or the reverse, change upstream and add binary node-findit to<br>
> > > > node-findit2.<br>
> > ><br>
> > ><br>
> > > What about this?<br>
> > ><br>
> > > * Add patch as quilt patch to node-findit that is already in debian<br>
> > > * When packaging other package updates that depend on findit2, patch them<br>
> > > to depend on node-findit instead, since the patch provides the findit2<br>
> > API<br>
> > > and bug fixes.<br>
> ><br>
> > I agree up to that point where you imply that findit2 has a different<br>
> > API ? If that's the case then findit2 must be... findit2 !<br>
> ><br>
><br>
> Hmm, in this case, since upstream seems to be ignoring my pull request,<br>
> maybe I will<br>
><br>
>  * rename the module to a better fork name (as I did with formidable -><br>
> multiparty)<br>
>  * add new node module to debian<br>
>  * update packages to depend on new module<br>
>  * delete node-findit from debian since nothing depends on it<br>
<br>
</div></div>If that's the case, then please keep a name close to the original one,<br>
so that when packaging a module depending on (old) findit, one can find<br>
easily the new findit to be a match.<br>
<br>
You might also want to commit to keep a backward-compatible API with<br>
node-findit.<br>
<br>
These are only suggestions.<span class="HOEnZb"><font color="#888888"></font></span></blockquote><div><br></div><div>I have thought about it, and I decided to keep the name as findit2. Also I agree to commit to backward-compatible API with node-findit, so that debian packages can use findit2 instead. I should have a package done here shortly. One thing that is kind of funny, is that since I cleaned up the code considerably, the lines of code is now small enough that FTP masters might say it is too small!! But hopefully if we tell them we are going to remove node-findit and this package is intended to replace it, then they will be OK with it. </div></div><br></div></div>