<div dir="ltr">Ahh, sorry, wrong Sebastian!<div><br></div><div>Gonzalo</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 25, 2014 at 9:51 AM, Sebastian Silva <span dir="ltr"><<a href="mailto:sebastian@somosazucar.org" target="_blank">sebastian@somosazucar.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">So ask Sebastian Dzallas why he removed my contribution!<br>
Sebastian Silva<br>
<a href="http://somosazucar.org/" target="_blank">http://somosazucar.org/</a><br>
<br>
<br>
<br>
2014-11-25 7:50 GMT-05:00 Gonzalo Odiard <<a href="mailto:godiard@sugarlabs.org">godiard@sugarlabs.org</a>>:<br>
<div class="HOEnZb"><div class="h5">> Yes, but that is what this patch do, right?<br>
><br>
> <a href="http://pkgs.fedoraproject.org/cgit/sugar-speak.git/commit/?id=438f3796a85556873c5e45d6ba01ccfa558ca198" target="_blank">http://pkgs.fedoraproject.org/cgit/sugar-speak.git/commit/?id=438f3796a85556873c5e45d6ba01ccfa558ca198</a><br>
><br>
> On Tue, Nov 25, 2014 at 9:47 AM, Sebastian Silva <<a href="mailto:sebastian@somosazucar.org">sebastian@somosazucar.org</a>><br>
> wrote:<br>
>><br>
>> btw it's sneaky to remove funcionality in a .spec file, no?<br>
>> Sebastian Silva<br>
>> <a href="http://somosazucar.org/" target="_blank">http://somosazucar.org/</a><br>
>><br>
>><br>
>><br>
>> 2014-11-25 7:46 GMT-05:00 Sebastian Silva <<a href="mailto:sebastian@somosazucar.org">sebastian@somosazucar.org</a>>:<br>
>> > I added Alice and Sara (spanish bot) but was not aware it was removed.<br>
>> > It certainly runs quite well in spanish XO1, as deployed in Peru.<br>
>> > I was not aware of responsible for the removed functionality nor was I<br>
>> > aware of any bugs related to it or performance.<br>
>> ><br>
>> > Please add it back<br>
>> > Sebastian Silva<br>
>> > <a href="http://somosazucar.org/" target="_blank">http://somosazucar.org/</a><br>
>> ><br>
>> ><br>
>> ><br>
>> > 2014-11-25 7:17 GMT-05:00 Peter Robinson <<a href="mailto:pbrobinson@gmail.com">pbrobinson@gmail.com</a>>:<br>
>> >> On Tue, Nov 25, 2014 at 12:05 PM, Gonzalo Odiard<br>
>> >> <<a href="mailto:godiard@sugarlabs.org">godiard@sugarlabs.org</a>> wrote:<br>
>> >>><br>
>> >>><br>
>> >>> On Tue, Nov 25, 2014 at 8:28 AM, Peter Robinson <<a href="mailto:pbrobinson@gmail.com">pbrobinson@gmail.com</a>><br>
>> >>> wrote:<br>
>> >>>><br>
>> >>>> On Mon, Nov 24, 2014 at 4:10 PM, Gonzalo Odiard<br>
>> >>>> <<a href="mailto:godiard@sugarlabs.org">godiard@sugarlabs.org</a>><br>
>> >>>> wrote:<br>
>> >>>> > You find it. I don't know why the bot directory should be deleted.<br>
>> >>>> > I cc to peter, who maintain the rpm.<br>
>> >>>><br>
>> >>>> It goes back long before I was  the maintainer and was previously<br>
>> >>>> working. The question is what does it do, and why when Speak has been<br>
>> >>>> working for years does it suddenly break.<br>
>> >>>><br>
>> >>>><br>
>> >>>><br>
>> >>>> <a href="http://pkgs.fedoraproject.org/cgit/sugar-speak.git/commit/?id=438f3796a85556873c5e45d6ba01ccfa558ca198" target="_blank">http://pkgs.fedoraproject.org/cgit/sugar-speak.git/commit/?id=438f3796a85556873c5e45d6ba01ccfa558ca198</a><br>
>> >>><br>
>> >>><br>
>> >>> That patch removes the bots functionality. Looks like was done to make<br>
>> >>> the<br>
>> >>> activity<br>
>> >>> work on systems with small memory (XO-1, probably)<br>
>> >>> cc is Sebastian, the patch author, to check if this is still needed.<br>
>> >>><br>
>> >>> I see 438f3796a85556873c5e45d6ba01ccfa558ca198 removes the bots<br>
>> >>> directory,<br>
>> >>> _and_ apply the patch  sugar-speak-no-aiml.patch, to remove the need<br>
>> >>> of the<br>
>> >>> bots data,<br>
>> >>> but right now, we are not applying that patch, then would be better<br>
>> >>> add the<br>
>> >>> bot directory again.<br>
>> >><br>
>> >> I suspect it was sufficiently long ago that Seb won't remember the<br>
>> >> reason.<br>
>> >><br>
>> >> Ultimately we need to just sort it out here.<br>
>> >><br>
>> >> Peter<br>
><br>
><br>
><br>
><br>
> --<br>
> Gonzalo Odiard<br>
><br>
> SugarLabs - Software for children learning<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Gonzalo Odiard<br><br><div>SugarLabs - Software for children learning <br></div></div></div>
</div>