<div dir="ltr">Sam sent a pull request to fix this, reviews would be appreciated<br><br><a href="https://github.com/sugarlabs/sugar/pull/333">https://github.com/sugarlabs/sugar/pull/333</a><br><div class="gmail_extra"><br>
</div><div class="gmail_extra">(You might have the honour to empty the review queue for the first time in a looong while).<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 17 February 2014 12:57, Daniel Narvaez <span dir="ltr"><<a href="mailto:dwnarvaez@gmail.com" target="_blank">dwnarvaez@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<div><br></div><div>parsing code to find that out is such an hack... </div><div><br></div><div>Anyway, I explained in the other thread what I think is the right way to do this in the future and I'm content with that. As long as</div>

<div><br></div><div>- the hack is done properly (no sync file loading)</div><div>- I'm not the one having to review the patch (too complicated to make it really solid)</div><div>- there is comment explaining that the proper way to do it would be to look at the exec field</div>

<div><br></div><div>Then I'm fine with it landing.</div><div class="HOEnZb"><div class="h5"><div><br>On Monday, 17 February 2014, Gonzalo Odiard <<a href="mailto:godiard@sugarlabs.org" target="_blank">godiard@sugarlabs.org</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">This is a reply to @dnarvaez comment <div><br></div><div><a href="https://github.com/sugarlabs/sugar-toolkit-gtk3/pull/104#issuecomment-35170356" target="_blank">https://github.com/sugarlabs/sugar-toolkit-gtk3/pull/104#issuecomment-35170356</a></div>


<div><br></div><div>The addition of a field to identify gtk2/gtk3 activities,</div><div>was proposed as a solution to View Source issue [1]</div><div><br></div><div>I think there are two other easier ways to solve this particular issue:</div>


<div><br></div><div>* check the setup.py file </div><div>* check the file with the startup class (view source already identify the file, see [2]</div><div><br></div><div>Making changes to the <a href="http://activity.info" target="_blank">activity.info</a> file usually take a long time </div>


<div>to get included in a significant number of activities, </div><div>if we can solve it in the code is better in my opinion.</div><div><br></div><div>-- <br></div><div><div><div dir="ltr">Gonzalo Odiard<br><br><div>SugarLabs - Learning Software for children<br>


</div><div><br></div><div>[1] <a href="https://github.com/sugarlabs/sugar-toolkit-gtk3/pull/104#issuecomment-35170356" target="_blank">https://github.com/sugarlabs/sugar-toolkit-gtk3/pull/104#issuecomment-35170356</a></div>

<div>[2] <a href="https://github.com/sugarlabs/sugar/blob/master/src/jarabe/view/viewsource.py#L205" target="_blank">https://github.com/sugarlabs/sugar/blob/master/src/jarabe/view/viewsource.py#L205</a></div>
</div>
</div></div></div>
</blockquote></div><br><br></div></div><span class="HOEnZb"><font color="#888888">-- <br>Daniel Narvaez<br><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>Daniel Narvaez<br>
</div></div>