<div dir="ltr">Is a good point.<div>Maybe we should add a dbus method to register a bundle,</div><div>and a method in the toolkit to do the call,</div><div>then this script use that method.</div><div>I didn't realized this before, because is included in the sugar module,</div><div>but we should not use jarabe here.</div><div><br></div><div>Gonzalo</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 31, 2015 at 2:03 PM, Martin Abente <span dir="ltr"><<a href="mailto:martin.abente.lahaye@gmail.com" target="_blank">martin.abente.lahaye@gmail.com</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">The problem caused by the race condition is that the activities were not refreshed in Sugar activities list when updated or, not added to the list when installed. By race condition I mean the situation where the BundleRegistry detects changes (e.g. in ~/Activities) too early, when <a href="http://activity.info" target="_blank">activity.info</a> is not available yet, so the bundle gets ignored. I personally don't think is related to this issue.<div><br></div><div>Now, looking at Tony's report and Sam's confirmation that this issue is related to the change in [1], my first guess would be that the problem is caused by the overhead generated by initializing the BundleRegistry, and any other sugar module imported by it, every time the sugar-install-bundle script is executed. Note that the script runs in a different process than the sugar shell, so the registry instance returned by get_registry() is not the same one as the one in the shell. Having SUGAR_PROFILE set doesn't necessarily mean the script is being executed in the same context as the shell.</div><div><br></div><div>If my guess is confirmed, we should revise that change or revert it.</div><div><br></div><div>Refs:</div><div>1. <a href="https://github.com/sugarlabs/sugar/commit/64b4b2fba1c37a9ad92ed30eb669b68552b62415" target="_blank">https://github.com/sugarlabs/sugar/commit/64b4b2fba1c37a9ad92ed30eb669b68552b62415</a><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 31, 2015 at 9:02 AM, Gonzalo Odiard <span dir="ltr"><<a href="mailto:godiard@gmail.com" target="_blank">godiard@gmail.com</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">I am wondering if this issue ca be related to the race condition detected<div>by Martin when the file monitor detect the activity before the <a href="http://activity.info" target="_blank">activity.info</a> file </div><div>is saved to disk.</div><div><br></div><div>Martin, do you have a patch to test?</div><div><br></div><div>Gonzalo</div></div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Mon, Aug 31, 2015 at 10:00 AM, Gonzalo Odiard <span dir="ltr"><<a href="mailto:godiard@gmail.com" target="_blank">godiard@gmail.com</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">Tony, <div>If you can enable logging (remove the comment in the line SUGAR_LOGGER_LEVEL</div><div>in the file /home/olpc/.sugar/default/debug)</div><div>And install one of the activities without unset the SUGAR_PROFILE env variable,</div><div>and send the log, maybe we can understand better what is happening.</div><div><br></div><div>I am afraid the slowness will impact Browse even if you patch it, because Browse process</div><div>run in a environment with the env variable set.</div><div><br></div><div>One more question: the time is bigger in slower devices (XO-1) than in XO-4,</div><div>or is independent of the hardware performance?</div><div><br></div><div>Gonzalo</div><div><br></div></div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Sun, Aug 30, 2015 at 7:34 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">G'day Sam,<br>
<br>
Adding a flag for a workaround seems like you've given up.  ;-)<br>
<br>
Wouldn't it be better to fix the problem?  As you can see from<br>
#4849 it occurs in Sugar, and only inside Terminal activity.<br>
<br>
Why is it happening?<br>
<br>
Why does it take up to 24 minutes to install Browse using Terminal?<br>
<br>
Why does it take up to 5 minutes to install in Sugar 0.98?<br>
<br>
And yet only half a second if the bundle registry is bypassed?<br>
<br>
The bundle registry can't be that slow.<br>
<br>
By understanding the problem we'll be able to say whether it is<br>
causing any other slowness of Sugar.<br>
<span><br>
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" rel="noreferrer" target="_blank">http://quozl.linux.org.au/</a><br>
</span><div><div>_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" rel="noreferrer" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br><div>Gonzalo Odiard<br><br></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span><font color="#888888">-- <br><div>Gonzalo Odiard<br><br></div>
</font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Gonzalo Odiard<br><br></div>
</div>