libiris: Stop using qt_wrap_cpp()

Authored by rakuco on Oct 13 2019, 1:53 PM.

Description

libiris: Stop using qt_wrap_cpp()

Summary:
The build system was currently both relying on CMake's AUTOMOC handling and
manually calling qt_wrap_cpp() for a few headers.

The former was causing the moc-generated code to go to
mocs_compilation.cpp, and the latter also caused the moc-generated
moc_*.cxx to be built. This resulted in the meta object code being built
twice, which is wrong, in the worst cause, cause linking problems.

Older LLD versions were affected per
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=228640

Drop qt_wrap_cpp() and include the missing moc files from the source code
and let CMake's automoc code handle everything transparently.

Test Plan:
Everything built fine, and CMake stopped emitting the following warning:

Policy CMP0071 is not set: Let AUTOMOC and AUTOUIC process GENERATED files.
Run "cmake --help-policy CMP0071" for policy details.  Use the cmake_policy
command to set the policy and suppress this warning.

For compatibility, CMake is excluding the GENERATED source file(s):

  "/wrkdirs/usr/ports/net-im/kopete/work/.build/protocols/jabber/libiris/moc_xmpp_client.cxx"
  "/wrkdirs/usr/ports/net-im/kopete/work/.build/protocols/jabber/libiris/moc_xmpp.cxx"
  "/wrkdirs/usr/ports/net-im/kopete/work/.build/protocols/jabber/libiris/moc_xmpp_clientstream.cxx"
  "/wrkdirs/usr/ports/net-im/kopete/work/.build/protocols/jabber/libiris/moc_xmpp_stream.cxx"
  "/wrkdirs/usr/ports/net-im/kopete/work/.build/protocols/jabber/libiris/moc_td.cxx"
  "/wrkdirs/usr/ports/net-im/kopete/work/.build/protocols/jabber/libiris/moc_qjdnsshared_p.cxx"
  "/wrkdirs/usr/ports/net-im/kopete/work/.build/protocols/jabber/libiris/moc_qjdns.cxx"
  "/wrkdirs/usr/ports/net-im/kopete/work/.build/protocols/jabber/libiris/moc_qjdnsshared.cxx"

from processing by AUTOMOC.  If any of the files should be processed, set
CMP0071 to NEW.  If any of the files should not be processed, explicitly
exclude them by setting the source file property SKIP_AUTOMOC:

  set_property(SOURCE file.h PROPERTY SKIP_AUTOMOC ON)

Reviewers: Kopete, Build System, apol

Reviewed By: apol

Subscribers: kopete-devel

Tags: Kopete

Differential Revision: https://phabricator.kde.org/D24609

Details

Committed
rakucoOct 18 2019, 7:18 PM
Reviewer
apol
Differential Revision
D24609: libiris: Stop using qt_wrap_cpp()
Parents
R434:22563ca5f946: Fix missing parentheses around macro
Branches
Unknown
Tags
Unknown