D10006: Build TerminalPanel also on Windows
Elvis Angelaccio
noreply at phabricator.kde.org
Sun Jan 21 11:26:42 GMT 2018
elvisangelaccio created this revision.
elvisangelaccio added reviewers: Dolphin, Craft.
elvisangelaccio requested review of this revision.
REVISION SUMMARY
terminalpanel.cpp *should* compile on Windows, so there is no reason to
remove it from the build and use tons of #ifdefs in dolphinmainwindow.
We still keep the terminal panel disabled on Windows
(i.e. the two remaining #ifndef Q_OS_WIN in dolphinmainwindow),
because it is probably not functional.
But at least we won't break the Windows CI every time
someone touches the terminal panel code (see e.g. https://phabricator.kde.org/R318:2e942237c977fc3fd7d3712167d859a8f56371e9).
TEST PLAN
Builds on Linux, someone on Windows should test this patch if possible.
REPOSITORY
R318 Dolphin
BRANCH
fix-terminalpanel-build (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D10006
AFFECTED FILES
src/CMakeLists.txt
src/dolphinmainwindow.cpp
src/dolphinmainwindow.h
To: elvisangelaccio, #dolphin, #craft
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180121/9358663c/attachment.htm>
More information about the kfm-devel
mailing list