<div dir="ltr">On 26 December 2013 17:50, Emil Dudev <span dir="ltr"><<a href="mailto:emildudev@gmail.com" target="_blank">emildudev@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Actually, when I started working on this, I changed the code in the<br>
following way:<br>
Read data from the GSettings database, write to both GSettings and GConf.<br>
But I changed my mind, when I saw the article. Guess I'll make an<br>
other commit, with my original idea.<br></blockquote><div><br></div><div>To be clear, I don't think we should do that for all the settings, only for the few used by activities.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

As noted above, the only activities I have, which use GConf are read,<br>
write and browse.<br>
For read and write I have made the necessary changes to drop GConf and<br>
I'll make the merge request in a minute.<br>
For browse, I have made only a small change. But there is an other problem.<br>
<br>
The problem with browse (and possibly, other activities) is that<br>
GSettings does not allow new keys to be created at runtime. Moreover,<br>
if sugar attempts to read a key, which doesn't exists, it will close.<br>
An exception won't be thrown, it won't hang up, it will just close<br>
with a single line in the log.<br>
This is a serious downfall for activities, which use custom keys.<br>
These keys have to be included in sugar's schema file, or sugar should<br>
allow activities to define their own schemas.<br></blockquote><div><br></div><div>Duuh, good catch.<br><br>We might also have issues in Sugar, I'm not too sure all the keys are in the schemas, we should verify that.<br>
<br>About activities, there is probably a way to make them ship schemas but I'm wondering what's the advantage for them to use gsettings vs a json or ini file. It feels like added complexity without real advantages.<br>
</div></div></div></div>