<br><br><div class="gmail_quote">On Mon, May 19, 2008 at 4:08 PM, Ryan P. Bitanga <<a href="mailto:ryan.bitanga@gmail.com">ryan.bitanga@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
2008/5/19 Aaron J. Seigo <<a href="mailto:aseigo@kde.org">aseigo@kde.org</a>>:<br>
<div class="Ih2E3d">> On Sunday 18 May 2008, Ryan P. Bitanga wrote:<br>
>> Anyway, attached is a preliminary patch to support a config dialog for<br>
>> KRunner.<br>
><br>
> hm .... this is really bad timing =) we are hoping to get a series of changes<br>
> in tomorrow to beat the feature freeze and have a branch going .. this will<br>
> need to be merged into the branch. =/<br>
<br>
</div>Well I was worried it wouldn't make it because I thought everything<br>
had to be fixed by May 17 and it's May 19 my time already.<br>
<div class="Ih2E3d"><br>
><br>
> also, we'll need to take scripting into consideration with the change to a kcm<br>
><br>
> KConfigGroup* RunnerManager::config() should be KConfigGroup<br>
> RunnerManager::config() (no pointer)<br>
><br>
> we'll need to allow for scripting somehow with regards to the configuration<br>
> now that it uses KCMs instead. hm... not sure how to do that off hand, but it<br>
> will need to get done.<br>
><br>
> i'm still not overly happy about the whitelist approach as it means every<br>
> newly installed runner will need to be added either by the user (not<br>
> desirable) or automatically with the right key in the config file ...<br>
> essentially working around the policy. what we really want is an opt-in<br>
> blacklist, not an opt-in whitelist.<br>
<br>
</div>Well changing it from a whitelist to a blacklist is as simple as<br>
changing a line in loadRunners() i.e. selected =<br>
!description.isPluginEnabled(). :) But in my opinion that would make<br>
it confusing.<br>
<br>
And I think it would be better if we just set<br>
X-KDE-PluginInfo-EnabledByDefault=true in the desktop files.<br>
<div class="Ih2E3d"></div></blockquote><div><br>I think we have argued about this at least two times before. As Aseigo has stated clearly his preference for blacklisting and you failed to convince us, it is too late for more arguing :P<br>
BTW, I've been missing lately, I don't expect to have much time till at least the release of 4.1.<br><br><br>A quick KActionSelector GUI can be hacked for the general configuration of RunnerManager, we are just out of schedule though... Left to 4.2?<br>
<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="Ih2E3d"><br>
> the big issue, however, is the timing of this. i'd say commit everything, but<br>
> please change the whitelisting approach. commit either within the next 12<br>
> hours so that a merge can be done with the branch or else hold off and wait<br>
> for 4.2. =( in fact here's what i'll do: if you don't get it committed by<br>
> noon my time i'll just commit it for you with some modifications so that i<br>
> can also merge it into the branch and be done with it.<br>
<br>
</div>I don't think I can commit it myself within 12 hours since my personal<br>
laptop is at home and I'm working late today. :( So please do commit<br>
it for me.<br>
<br>
Cheers,<br>
Ryan<br>
_______________________________________________<br>
Panel-devel mailing list<br>
<a href="mailto:Panel-devel@kde.org">Panel-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/panel-devel" target="_blank">https://mail.kde.org/mailman/listinfo/panel-devel</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>Jordi Polo Carres<br>NLP laboratory - NAIST<br><a href="http://www.bahasara.org">http://www.bahasara.org</a><br>