cgiboudeaux added a comment.
In D15070#344871 <https://phabricator.kde.org/D15070#344871>, @bruns wrote: > As all the raised concerns have been dealed with, can we give this a try while the next KF release is still somewhat in the future? No, the empty if must be removed. Code that does nothing is useless. REPOSITORY R240 Extra CMake Modules REVISION DETAIL https://phabricator.kde.org/D15070 To: bruns, #frameworks Cc: cgiboudeaux, bcooksley, kde-frameworks-devel, kde-buildsystem, michaelh, ngraham, bruns