<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Aug 9, 2014 at 2:07 PM, Ben Cooksley <span dir="ltr"><<a href="mailto:bcooksley@kde.org" target="_blank">bcooksley@kde.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Git commit c8b445b04a0efe66e491a6f0a06ca4ef312fdc91 by Ben Cooksley.<br>
Committed on 09/08/2014 at 12:00.<br>
Pushed by bcooksley into branch 'master'.<br>
<br>
Revert "Fix plugins lookup when not building in the Qt prefix"<br>
<br>
This reverts commit 38e0665127650b3db5676c1dd5f0ffeb8cdb176f.<br>
<br>
This change is incorrect, as Qt/QCA does not look within $pluginDir/qca/crypto when loading it's plugins.<br>
Instead, it expects to find them within the $pluginDir/crypto/ subdirectory which no longer exists with this change.<br>
<br>
While the use of lib/qca/ is non-standard for plugins, existing code within the CI scripts is able to handle this abnormality.<br>
Normal Qt libraries which use plugins also function normally on the CI system.<br>
<br>
The change as it stands does not work with Qt 4 and breaks all tests which are reliant upon QCA on the CI system.<br>
As it appears a decision needs to be made on where QCA plugins should live and how they should be loaded i'm reverting this change to allow the tests to pass again.<br>
<br>
CCMAIL: <a href="mailto:aleixpol@kde.org">aleixpol@kde.org</a><br>
CCMAIL: <a href="mailto:drizt@land.ru">drizt@land.ru</a><br>
CCMAIL: <a href="mailto:stephane@mankowski.fr">stephane@mankowski.fr</a><br>
CCMAIL: <a href="mailto:kde-core-devel@kde.org">kde-core-devel@kde.org</a><br>
CCMAIL: <a href="mailto:kde-frameworks-devel@kde.org">kde-frameworks-devel@kde.org</a><br>
<br>
M  +1    -1    CMakeLists.txt<br>
<br>
<a href="http://commits.kde.org/qca/c8b445b04a0efe66e491a6f0a06ca4ef312fdc91" target="_blank">http://commits.kde.org/qca/c8b445b04a0efe66e491a6f0a06ca4ef312fdc91</a><br>
<br>
diff --git a/CMakeLists.txt b/CMakeLists.txt<br>
index a4dc6de..24d8ab3 100644<br>
--- a/CMakeLists.txt<br>
+++ b/CMakeLists.txt<br>
@@ -158,7 +158,7 @@ else( QCA_INSTALL_IN_QT_PREFIX )<br>
   set(LIB_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}/lib${LIB_SUFFIX}" CACHE PATH "Directory where lib will install")<br>
<br>
   set(QCA_PREFIX_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}" CACHE PATH "Directory where qca will install")<br>
-  set(QCA_PLUGINS_INSTALL_DIR "${LIB_INSTALL_DIR}/plugins/${QCA_LIB_NAME}/crypto" CACHE PATH "Directory where qca plugins will install")<br>
+  set(QCA_PLUGINS_INSTALL_DIR "${LIB_INSTALL_DIR}/${QCA_LIB_NAME}/crypto" CACHE PATH "Directory where qca plugins will install")<br>
   set(QCA_BINARY_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}/bin" CACHE PATH "Directory where qca plugins will install")<br>
   set(QCA_LIBRARY_INSTALL_DIR "${LIB_INSTALL_DIR}" CACHE PATH "Directory where qca library will install")<br>
   set(QCA_FEATURE_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}/mkspecs/features" CACHE PATH "Directory where qca feature file will install")<br>
</blockquote></div><br></div><div class="gmail_extra">My bad, I didn't try on Qt4, only assumed it would be the same as Qt5.<br><br>Either way, this breaks Qt5 now.<br><br>Aleix<br></div></div>