<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2016-10-04 13:04 GMT+02:00 Tanu Kaskinen <span dir="ltr"><<a href="mailto:tanuk@iki.fi" target="_blank">tanuk@iki.fi</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, 2016-09-30 at 23:21 +0200, Sylvain Baubeau wrote:<br>
> 2016-09-27 21:32 GMT+02:00 Felipe Sateler <<a href="mailto:fsateler@debian.org">fsateler@debian.org</a>>:<br>
</span><span class="">> > I have to say I'm not sure how an upgrade scenario would play out. My<br>
> > understanding currently is as follows:<br>
> ><br>
> > 1. paprefs can ship a file for gsettings-data-convert, which migrates<br>
> > the settings.<br>
> ><br>
> > 2. However that tool is only run at login.<br>
> ><br>
> > 3. As a consequence, the paprefs tool can be in an inconsistent state<br>
> > after an upgrade and before the next login.<br>
> ><br>
> > It seems that the problem is resolved by papfrefs triggering the tool<br>
> > on startup as suggested by the gnome documentation[1].<br>
> ><br>
><br>
> Thanks for pointing this out. I updated my paprefs patch to apply<br>
> paprefs.convert at startup.<br>
<br>
</span>Would it be feasible for you to make it a build time option to use<br>
gconf or gsettings in paprefs?<br></blockquote><div><br></div><div>Yes, I think it's feasible but the code would need quite a lot of rework.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> > However that leaves us with another problem, as the pulseaudio daemon<br>
> > is not restarted automatically after an upgrade, and thus gsettings<br>
> > module might not be loaded. But, I hope paprefs can already deal with<br>
> > such a scenario and prompt the user accordingly.<br>
> ><br>
><br>
> Unfortunately, paprefs does not deal with such a scenario. It will<br>
> therefore be able to modify the configuration but this configuration will<br>
> only take effect when pulseaudio is restarted. What do you suggest<br>
> prompting the user with ?<br>
<br>
</span>Figuring out a good message is tricky. paprefs could just say "module-<br>
gsettings not loaded", but it would be nice to able to tell the user<br>
what to do. However, module-gsettings not being loaded may be because<br>
the server is too old version and doesn't support module-gsettings at<br>
all (distributions can deal this by depending on new enough server,<br>
though), or it has been removed from <a href="http://default.pa" rel="noreferrer" target="_blank">default.pa</a> and needs to be added<br>
there, or the server has been updated but not restarted. The last<br>
reason is the most likely one in the beginning, but over time it<br>
becomes less and less likely, so just saying "restart pulseaudio" or<br>
"reboot the machine" probably isn't a good idea.<br>
<span class=""><br>
> > In debian there is only paprefs depending on module-gconf. I don't<br>
> > think we'll keep module-gconf after paprefs has migrated.<br>
<br>
</span>If we release a pulseaudio version with module-gsettings before we<br>
release paprefs with gsettings support, will you run pulseaudio with<br>
both gconf and gsettings enabled, or only with gconf? Running with both<br>
would have the benefit that if everything goes well, the data will be<br>
migrated without any glitch when the new paprefs version is run for the<br>
first time, because module-gsettings will be already loaded. The risk<br>
with that is that the migration can't be tested at the time you enable<br>
module-gsettings, since paprefs isn't ready. If there's some unforeseen<br>
problem when paprefs is finally updated, and module-gsettings needs to<br>
be fixed in some way, the migration might become even more complex,<br>
because you need to deal with detecting a situation where the server is<br>
running with a broken version of module-gsettings.<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>You're right, it's probably too complicated. It's better to stick with GConf.</div><div><br>Regards,</div><div>Sylvain</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
-- <br>
Tanu<br>
</font></span><div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
pulseaudio-discuss mailing list<br>
<a href="mailto:pulseaudio-discuss@lists.freedesktop.org">pulseaudio-discuss@lists.<wbr>freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/pulseaudio-<wbr>discuss</a><br>
</div></div></blockquote></div><br></div></div>