conan-io / conan-center-index

Recipes for the ConanCenter repository
https://conan.io/center
MIT License
970 stars 1.78k forks source link

[package] qt/6.7.3: Missing Metal Framework when consuming qtGUI module #26005

Open uilianries opened 3 days ago

uilianries commented 3 days ago

Description

When consuming Qt 6.7, the same has a specific rule for Macos and iOS in the upstream CMake file:

https://github.com/qt/qtbase/blob/v6.7.3/src/gui/CMakeLists.txt#L463

When the graphics frame capture feature is enabled, then qtGui should link to Metal framework as well.

The current recipe in CCI has no such rule for qtGui: https://github.com/conan-io/conan-center-index/blob/master/recipes/qt/6.x.x/conanfile.py#L1193

It applies Metal framework, but only for QIOSIntegrationPlugin and QCocoaIntegrationPlugin.

Package and Environment Details

Conan profile

[settings] arch=armv8 build_type=Release compiler=apple-clang compiler.cppstd=17 compiler.libcxx=libc++ compiler.version=16 os=Macos [options] &:shared=True [conf] tools.build:verbosity=verbose tools.cmake.cmaketoolchain:generator=Ninja tools.compilation:verbosity=verbose

Steps to reproduce

1) Fetch the PR for Node Editor: https://github.com/conan-io/conan-center-index/pull/24751 2) Checkout the commit https://github.com/conan-io/conan-center-index/pull/24751/commits/f0b8eb7468d6747ea65ce142d4f1a105b3ff516b 3) On Mac M1, build the library using option "&:shared=True"

In case using all static, it should fail when linking the test package too.

Logs

nodeeditor-3.0.11-macos-shared.log