<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 01/05/2014 09:57 AM, Hugo Pereira Da
Costa wrote:<br>
</div>
<blockquote cite="mid:52C91E90.2080800@free.fr" type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<div class="moz-cite-prefix">On 01/05/2014 09:42 AM, Martin
Graesslin wrote:<br>
</div>
<blockquote cite="mid:1546316.YRf7ScPhdh@martin-thinkpad"
type="cite">
<pre wrap="">On Saturday 04 January 2014 20:52:43 David Faure wrote:
</pre>
<blockquote type="cite">
<pre wrap="">On Saturday 04 January 2014 20:19:43 Hugo Pereira Da Costa wrote:
</pre>
<blockquote type="cite">
<pre wrap="">The first problem is that with kf5, the style configuration kcm expects
a libkstyle_oxygen_config.so *plugin* for this configuration, whereas it
was "kstyle_oxygen_config.so" in kde4 times.
now, as the error message indicates: plugins should not start with "lib"
and thus, oxygen still installs kstyle_oxygen_config.so only
so that the ill-named plugin is not found, and you get the error dialog.
To fix,
- either we fix the kcm by changing the looked for plugin name (by
removing the lib prefix)
</pre>
</blockquote>
<pre wrap="">Yes. Why does the kcm look for a lib prefix now? Surely there's no good
reason for that, the kde4 way was fine.
</pre>
</blockquote>
<pre wrap="">it was me who ported the kcm. I didn't change anything in that regard - at
least not knowingly. The only change in KCMStyle::styleSpecificConfig() was:
- KLibrary library(libname, KCMStyleFactory::componentData());
+ KLibrary library(libname);
Also I think to remember that it used to work when I did the port. So I rather
think it got broken by a3194a6010961c68da80a9a8aa97501fb637f4a1
</pre>
</blockquote>
This commit is the one that _removed_ the lib prefix, consistently
with what was done with kde4:<br>
(see /usr/lib64/kde4/kstyle_oxygen_config.so, and now:
$KF5/lib/plugins/kf5/kstyle_oxygen_config.so<br>
all consistent !)<br>
<br>
as I said, this commit did break the kf5 style kcm, that
(incorrectly) expects a lib prefix, and is therefore the one that
shows up by git dissect, but I think we'd rather fix the "expect a
lib prefix part" rather than revert the commit.<br>
<br>
Best,<br>
<br>
</blockquote>
mmm. My bad. In fact the current breakage is an inconsistency
between what oxygen installs (kstyle_oxygen_config) and what it
announce in oxygen.themerc.<br>
Just fixed it. Sorry for the noise. Nothing was changed indeed to
the kstyle kcm.<br>
<br>
Hugo<br>
<blockquote cite="mid:52C91E90.2080800@free.fr" type="cite"> Hugo<br>
</blockquote>
<br>
<blockquote cite="mid:52C91E90.2080800@free.fr" type="cite">
<blockquote cite="mid:1546316.YRf7ScPhdh@martin-thinkpad"
type="cite">
<pre wrap="">Cheers
Martin</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
Kde-frameworks-devel mailing list
<a moz-do-not-send="true" class="moz-txt-link-abbreviated" href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a>
<a moz-do-not-send="true" class="moz-txt-link-freetext" href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a>
</pre>
</blockquote>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
Kde-frameworks-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a>
<a class="moz-txt-link-freetext" href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a>
</pre>
</blockquote>
<br>
</body>
</html>