Check for HAVE_X11 instead of Q_OS_LINUX in bringToFront

CMakeLists.txt

@@ -159,9 +159,14 @@ if( NOT BUILD_WITH_QT4 )
         find_package(Qt5WebKitWidgets REQUIRED)
         find_package(Qt5Widgets REQUIRED)
         find_package(Qt5Xml REQUIRED)
-        if(UNIX AND NOT APPLE)
+        find_package(Qt5X11Extras NO_MODULE)
-            find_package(Qt5X11Extras REQUIRED NO_MODULE)
+
+        if(Qt5X11Extras_FOUND)
+            set(HAVE_X11 TRUE)
+        else()
+            set(HAVE_X11 FALSE)
         endif()
+
         message(STATUS "Found Qt5! Be aware that Qt5-support is still experimental and not officially supported!")
 
         if( UNIX AND NOT APPLE )

src/libtomahawk/utils/TomahawkUtilsGui.cpp

@@ -41,17 +41,15 @@
 #include <QDesktopServices>
 
 #if QT_VERSION >= QT_VERSION_CHECK( 5, 0, 0 )
-#ifdef Q_OS_LINUX
+    #ifdef HAVE_X11
-    #include <QWindow>
+        #include <QWindow>
-    #include <QX11Info>
+        #include <QX11Info>
-#endif
+    #endif
-#endif
+#else
-
+    #ifdef Q_WS_X11
-
+        #include <QX11Info>
-//FIXME: Qt5: this doesnt fail because Q_WS_X11 is deprecated
+        #include <libqnetwm/netwm.h>
-//TODO: change to Q_OS_X11 and fix errors
+    #endif
-#ifdef Q_WS_X11
-    #include <libqnetwm/netwm.h>
 #endif
 
 #ifdef Q_OS_WIN
@@ -356,8 +354,7 @@ bringToFront()
         }
     }
 #endif
-#endif
+#else // Qt5
-#if QT_VERSION >= QT_VERSION_CHECK( 5, 0, 0 )
     {
         qDebug() << Q_FUNC_INFO;
 

src/tomahawk/Config.h.in

@@ -25,5 +25,6 @@
 #cmakedefine TOMAHAWK_FINEGRAINED_MESSAGES
 #cmakedefine COMPLEX_TAGLIB_FILENAME
 #cmakedefine HAVE_VLC_ALBUMARTIST
+#cmakedefine HAVE_X11
 
 #endif // CONFIG_H_IN