<div dir="ltr">Hey Sam,<div><br></div><div>I checked your PR but I think I found a simpler way [1] to handle this. The logic goes like this:</div><div><br></div><div>1. assume you have tubes.</div><div>2. if everything goes OK, then you do have tubes.</div><div>3. but, if tubes creation fails, and the reason is lack of support, then you don't have tubes (sort of detection), and let process to continue.</div><div><br></div><div>This achieves the exact same thing, and does not require new config values.</div><div><br></div><div>What do you think?</div><div><br></div><div>Martin.</div><div><br></div><div>Refs</div><div>[1] <a href="https://github.com/sugarlabs/sugar-toolkit-gtk3/pull/297">https://github.com/sugarlabs/sugar-toolkit-gtk3/pull/297</a></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 28, 2015 at 6:05 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"><div class="HOEnZb"><div class="h5">On Mon, Dec 28, 2015 at 02:37:20PM -0300, Martin Abente wrote:<br>
> @Sam:<br>
><br>
>   • Using GSettings is definitely a decent workaround instead of going for<br>
>     detection, I checked your PR and I will send some comments, but we will<br>
>     have to push it for as a feature freeze exception / Bug Fix.<br>
>   • I don't agree we should add it to the toolkit yet, still too early to set<br>
>     API on stones, but lets make a good use of that Wrapper of yours. Thanks<br>
>     for making it available.<br>
>   • Collaboration should be transparent for Activities, and since Tubes is no<br>
>     longer available in at least one scenario (jabber), we can't really<br>
>     depending on it anymore.<br>
><br>
> @Tony:<br>
><br>
>   • As Sam explained, this is not something _we_ control. Telepathy dropped<br>
>     Tubes years ago. There is no way to "workaround" the fact that Tubes in<br>
>     gone and we gotta move to something else, but for now we will also keep<br>
>     Tubes for those who can still use it and at least be able to do the<br>
>     transition.<br>
><br>
> @Gonzalo:<br>
><br>
>   • Thanks for the feedback, you are right, collaboration is hard to test, so<br>
>     we can't rush here.<br>
><br>
> @James C.:<br>
><br>
>   • Telepathy developers knew Sugar was using still it, I even heard they<br>
>     warned us in a GUADEC conference years ago, I also heard they apologized in<br>
>     advanced.<br>
>   • I am against introducing one deprecated piece of Telepathy code into Sugar<br>
>     code, simply because we can barely catch up with our own code base and it<br>
>     would be even worse if we adopt that code.<br>
>   • Aside from that, Walter and Sam tests show that there are many cases where<br>
>     we can simply migrate other channels. It will take longer, but is the right<br>
>     investment IMHO.<br>
>   • The Tubes issue is just as critical as any other "we can't catch up with<br>
>     upstream" issue but, this time, we are trying not to ignore it.<br>
>   • Any comments regarding the use of a GSettings entry to re-enable Tubes<br>
>     support (and leave it disabled by default)?<br>
>   • Funny picture.<br>
<br>
</div></div>No worries, thanks for taking the decisions.<br>
<span class="im HOEnZb"><br>
><br>
> @Walter:<br>
><br>
>   • I agree, lets move forward.<br>
><br>
> @Jonas:<br>
><br>
>   • I hope so too.<br>
><br>
> Martin.<br>
<br>
</span><div class="HOEnZb"><div class="h5">--<br>
James Cameron<br>
<a href="http://quozl.netrek.org/" rel="noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
</div></div></blockquote></div><br></div>