Skip to content

build: Getting ready to Qt 6 (4/n). Improve `build-aux/m4/bitcoin_qt.m4`

Placeholder Hennadii Stepanov requested to merge github/fork/hebasto/220409-appcheck into master

This PR adds a test macro _BITCOIN_QT_CHECK_APP, which ensures that Qt resource system object (*.rcc) have being compiled and linked properly. Found it such checks very useful while working on bitcoin/bitcoin#24798.

Here are examples from configure logs:

  • successful check:
...
checking for Qt5Core >= 5.11.3... yes
checking for QCoreApplication initialization... yes
checking for Qt5Gui >= 5.11.3... yes
checking for QGuiApplication initialization... yes
checking for Qt5Widgets >= 5.11.3... yes
checking for QApplication initialization... yes
checking for Qt5Network >= 5.11.3... yes
checking for Qt5Test >= 5.11.3... yes
checking for Qt5DBus >= 5.11.3... yes
...
  • failed check:
...
checking for QCoreApplication initialization... no
configure: WARNING: QCoreApplication failed to initialize.; bitcoin-qt frontend will not be built
checking whether to build Bitcoin Core GUI... no
...

The fourth commit may fix build in some weird setups, but it is definitely required for Qt 6.

Merge request reports