<div dir="ltr">I have been testing and reviewing this PR since the last week and I think it would be beneficial for 0.102.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 5, 2014 at 6:33 PM, Gonzalo Odiard <span dir="ltr"><<a href="mailto:godiard@sugarlabs.org" target="_blank">godiard@sugarlabs.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Is a good idea as plan B :)</div><div class="gmail_extra"><div class=""><br><br><div class="gmail_quote">
On Mon, May 5, 2014 at 7:25 PM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I don't think I qualify as a developer; no recent coding.<br>
<br>
I'm in favour of your change, as it is not a public library ABI break,<br>
but an enhancement; compatibility is retained.<br>
<br>
However, if you can't get traction, rather than enhance the ABI,<br>
duplicate the new mime.py implementation into<br>
src/jarabe/journal/model.py in pr 367 for the release, and for next<br>
version move it from model.py to mime.py.<br>
<br>
This will fix the bug, avoid ABI change, and generate more work later.<br>
<span><font color="#888888"><br>
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div></div><div class="">-- <br><div dir="ltr">Gonzalo Odiard<br><br><div>SugarLabs - Software for children learning <br></div></div>
</div></div>
<br>_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
<br></blockquote></div><br></div>