<div dir="ltr">Hey,<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 6, 2015 at 7:56 PM, Marko Käning <span dir="ltr"><<a href="mailto:mk-lists@email.de" target="_blank">mk-lists@email.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi devs,<br>
<br>
kglobalaccel is currently failing on Jenkins [1].<br>
<br>
<br>
This can partially be fixed by inserting some dependencies for it:<br>
---<br>
diff --git a/dependency-data-kf5-qt5 b/dependency-data-kf5-qt5<br>
index 55b4141..91ac810 100644<br>
--- a/dependency-data-kf5-qt5<br>
+++ b/dependency-data-kf5-qt5<br>
@@ -16,6 +16,13 @@<br>
 *: Qt5[stable]<br>
 *: kdesupport/extra-cmake-modules<br>
<br>
+# Frameworks, tier1<br>
+frameworks/kglobalaccel: frameworks/kconfig<br>
+frameworks/kglobalaccel: frameworks/kcoreaddons<br>
+frameworks/kglobalaccel: frameworks/kcrash<br>
+frameworks/kglobalaccel: frameworks/kdbusaddons<br>
+frameworks/kglobalaccel: frameworks/ki18n<br>
+<br>
 # Frameworks, tier2<br>
 frameworks/kauth: frameworks/kcoreaddons<br>
 frameworks/kcompletion: frameworks/kconfig<br>
---<br></blockquote><div><br></div><div>Ahh yes, I prepared this and then got carried away by other work, this needs to be done.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
... yet I wonder whether this is actually wanted!?! I thought that tier 1<br>
frameworks won’t depend on any other framework, but kglobalaccel now seems<br>
to be the 1st framework not being in correspondence with this!<br></blockquote><div><br></div><div>KGlobalAccel got the runtime daemon added into the framework today</div><div>and is now tier3 (starting from 5.7 release).</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
But well, even with the above lines I run into this after all:<br>
---<br>
/Users/marko/WC/KDECI-builds/kglobalaccel/src/runtime/globalshortcutsregistry.cpp:38:10: fatal error: 'kglobalaccel_qws.h' file not found<br>
#include "kglobalaccel_qws.h"<br>
         ^<br>
1 error generated.<br>
make[2]: *** [src/runtime/CMakeFiles/kglobalaccel5.dir/globalshortcutsregistry.cpp.o] Error 1<br>
---<br></blockquote><div><br></div><div>Ah humm..that would be my fault, builds fine here. I'll have a look.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This seems to be caused by the use of Q_WS_MACX. Replacing it by Q_OS_MAC in<br>
src/runtime/globalshortcutsregistry.cpp makes it build the OSX code path:<br>
---<br>
diff --git src/runtime/globalshortcutsregistry.cpp src/runtime/globalshortcutsregistry.cpp<br>
index 532334a..3f84edb 100644<br>
--- src/runtime/globalshortcutsregistry.cpp<br>
+++ src/runtime/globalshortcutsregistry.cpp<br>
@@ -30,7 +30,7 @@<br>
<br>
 #if HAVE_X11<br>
 #include "kglobalaccel_x11.h"<br>
-#elif defined(Q_WS_MACX)<br>
+#elif defined(Q_OS_MAC)<br>
 #include "kglobalaccel_mac.h"<br>
 #elif defined(Q_WS_WIN)<br>
 #include "kglobalaccel_win.h"<br>
---<br>
<br>
Compiling now gets further, but unfortunately fails:<br>
---<br>
<br>
In file included from /Users/marko/WC/KDECI-builds/kglobalaccel/src/runtime/globalshortcutsregistry.cpp:34:<br>
/Users/marko/WC/KDECI-builds/kglobalaccel/src/runtime/kglobalaccel_mac.h:25:10: fatal error: 'kshortcut.h' file not found<br>
#include "kshortcut.h"<br>
         ^<br>
---<br>
<br>
Ideas?<br></blockquote><div><br></div><div>I'll have a look and fix it.</div><div><br></div><div>Thanks for the investigation!</div></div><div><br></div><div>Cheers</div>-- <br><div class="gmail_signature"><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div></div>
</div></div>