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><br>On Monday, 17 February 2014, Gonzalo Odiard <<a href="mailto:godiard@sugarlabs.org">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>-- <br>Daniel Narvaez<br><br>