Don't build qwindowsurface autotest on Mac OS.
authorJason McDonald <jason.mcdonald@nokia.com>
Tue, 4 Oct 2011 06:58:55 +0000 (16:58 +1000)
committerQt by Nokia <qt-info@nokia.com>
Thu, 6 Oct 2011 03:28:32 +0000 (05:28 +0200)
The test was a no-op on Mac OS. It's better to avoid building or running
it on Mac OS, rather than spending time compiling and running a test
that does nothing on that platform.

Change-Id: I4aa579036788e404f8a6a8f1a80acd32338fb3cd
Reviewed-on: http://codereview.qt-project.org/5958
Reviewed-by: Qt Sanity Bot <qt_sanity_bot@ovi.com>
Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
tests/auto/gui.pro
tests/auto/qwindowsurface/tst_qwindowsurface.cpp

index 9d7adc6..180cf04 100644 (file)
@@ -135,6 +135,9 @@ SUBDIRS=\
     qwswindowsystem \
     qx11info \
 
+# This test cannot be run on Mac OS
+mac*:SUBDIRS -= qwindowsurface
+
 # This test takes too long to run on IRIX, so skip it on that platform
 irix-*:SUBDIRS -= qitemview
 
index 5fdd273..fb273a0 100644 (file)
@@ -45,7 +45,7 @@
 #include <QPalette>
 #include <QWindowsStyle>
 
-#if !defined(Q_WS_MAC) && QT_VERSION < 0x050000 // Fixme Qt 5
+#if QT_VERSION < 0x050000 // Fixme Qt 5
 
 #include <private/qwindowsurface_p.h>
 #include <QDesktopWidget>
@@ -283,8 +283,9 @@ void tst_QWindowSurface::grabWidget()
 
 QTEST_MAIN(tst_QWindowSurface)
 
-#else // Q_WS_MAC
+#else
 
+// Temporarily skip the entire test until it is fixed for Qt5
 QTEST_NOOP_MAIN
 
 #endif