On August 5th, 2012, 9:07 a.m., David Faure wrote:

Ah, yes, this is a porting error. KDE4 cmake stuff used to define Q_WS_* cmake variables, but nothing does that in KF5.

I trust you that "WIN32" and "APPLE" are right here, no experience with that.

However this should be fixed across the board, not just in kidletime:

kdecore/CMakeLists.txt:if (NOT Q_OS_X11)    // LOL that's a good one :-)
kded/CMakeLists.txt:if (Q_OS_MAC)
kded/CMakeLists.txt:endif (Q_OS_MAC)
kdeui/CMakeLists.txt:if (Q_OS_MAC AND MAC_USE_OSXKEYCHAIN)
kdeui/CMakeLists.txt:else(Q_OS_MAC AND MAC_USE_OSXKEYCHAIN)
kdeui/CMakeLists.txt:else(Q_OS_MAC AND MAC_USE_OSXKEYCHAIN)
kdeui/CMakeLists.txt:endif(Q_OS_MAC AND MAC_USE_OSXKEYCHAIN)
kdeui/CMakeLists.txt:if (Q_OS_MAC)
kdeui/CMakeLists.txt:endif (Q_OS_MAC)
kdeui/CMakeLists.txt:if (Q_OS_WIN)
kdeui/CMakeLists.txt:endif (Q_OS_WIN)
kdeui/CMakeLists.txt:if(Q_OS_MAC)
kdeui/CMakeLists.txt:endif(Q_OS_MAC)
kdeui/CMakeLists.txt:if (Q_OS_MAC)
kdeui/CMakeLists.txt:elseif (Q_OS_WIN)
kdeui/CMakeLists.txt:else(Q_OS_WIN)
kdeui/CMakeLists.txt:endif (Q_OS_MAC)
kdeui/CMakeLists.txt:if (Q_OS_MAC)
kdeui/CMakeLists.txt:endif (Q_OS_MAC)
kinit/CMakeLists.txt:if (Q_OS_MAC)
kinit/CMakeLists.txt:else (Q_OS_MAC)
kinit/CMakeLists.txt:endif (Q_OS_MAC)
tier1/kidletime/src/CMakeLists.txt:elseif (Q_OS_MAC)
tier1/kidletime/src/CMakeLists.txt:elseif (Q_OS_WIN)
tier1/kwindowsystem/src/CMakeLists.txt:if (Q_OS_MAC)
tier1/kwindowsystem/src/CMakeLists.txt:if (Q_OS_WIN)

On August 5th, 2012, 2:57 p.m., Alexander Neundorf wrote:

Yes, this was set by FindQt4.cmake.
But, AFAIK with Lighthouse this is not the case anymore.
if(WIN32) is also true for cygwin until cmake 2.8.3, since 2.8.4 it is false under cygwin by default.

But is it okay to keep if(WIN32) considering that cygwin is not supported?

- Andrius da Costa

On August 4th, 2012, 9:57 p.m., Andrius da Costa Ribas wrote:

Q_OS_WIN & Q_OS_MAC not defined at that point, replacing with WIN32 and APPLE.

Got undefined references before the fix.