mingw-w64-x86_64-qgpgme-qt5: use of IMPORTED_IMPLIB_RELEASE breaks non-release CMake builds

[amyspark]

Description / Steps to reproduce the issue

1. Install mingw-w64-x86_64-qgpgme-qt5.
2. Do a find_package(QGpgme) on any project.
3. Link against the QGpgme target.

Expected behavior

Link should succeed.

Actual behavior

[proc] Ejecutando el comando: "C:\Program Files\CMake\bin\cmake.EXE" --build e:/kmymoney/build --config RelWithDebInfo --target install --
[build] ninja: error: 'kmymoney/misc/QGpgme-NOTFOUND', needed by 'kmymoney/misc/CMakeFiles/kmm_gpgfile_autogen_timestamp_deps', missing and no known rule to make it

This is because the import library as provided by MSYS2 is qualified for only the Release build type.

Verification

• I have verified that my MSYS2 is up-to-date before submitting the report (see https://www.msys2.org/docs/updating/)

Windows Version

MINGW64_NT-10.0-19045

MINGW environments affected

• MINGW64
• MINGW32
• UCRT64
• CLANG64
• CLANGARM64

Are you willing to submit a PR?

As soon as I get confirmation from a maintainer

[Biswa96]

Please provide steps to reproduce the issue. Also, the cmake does not seem from msys2.

[amyspark]

CMakeLists.txt:

cmake_minimum_required(VERSION 3.19)
project(gpgme2 C CXX)

find_package(Qt5 COMPONENTS Core REQUIRED)
find_package(QGpgme REQUIRED)

add_executable(main main.cpp)

target_link_libraries(main PUBLIC QGpgme)

main.cpp:

int main() {
    return 0;
}

Configuring with a recent enough CMake detects the error:

[main] Configurando el proyecto: gpgme 
[proc] Ejecutando el comando: D:/msys64/clang64/bin/cmake.exe -DCMAKE_BUILD_TYPE:STRING=Debug -DCMAKE_INSTALL_PREFIX:STRING=c:/Users/Amalia/Desktop/Rachel/gpgme/install -DCMAKE_EXPORT_COMPILE_COMMANDS:BOOL=TRUE -DCMAKE_CXX_COMPILER:FILEPATH=D:/msys64/clang64/bin/clang++.exe -DCMAKE_CC_COMPILER:FILEPATH=D:/msys64/clang64/bin/clang.exe --no-warn-unused-cli -SC:/Users/Amalia/Desktop/Rachel/gpgme -Bc:/Users/Amalia/Desktop/Rachel/gpgme/build -G Ninja
[cmake] Not searching for unused variables given on the command line.
[cmake] -- The C compiler identification is Clang 18.1.8
[cmake] -- The CXX compiler identification is Clang 18.1.8
[cmake] -- Detecting C compiler ABI info
[cmake] -- Detecting C compiler ABI info - done
[cmake] -- Check for working C compiler: D:/msys64/clang64/bin/cc.exe - skipped
[cmake] -- Detecting C compile features
[cmake] -- Detecting C compile features - done
[cmake] -- Detecting CXX compiler ABI info
[cmake] -- Detecting CXX compiler ABI info - done
[cmake] -- Check for working CXX compiler: D:/msys64/clang64/bin/clang++.exe - skipped
[cmake] -- Detecting CXX compile features
[cmake] -- Detecting CXX compile features - done
[cmake] -- Configuring done (3.8s)
[cmake] CMake Warning (dev) in CMakeLists.txt:
[cmake]   Policy CMP0111 is not set: An imported target missing its location property
[cmake]   fails during generation.  Run "cmake --help-policy CMP0111" for policy
[cmake]   details.  Use the cmake_policy command to set the policy and suppress this
[cmake]   warning.
[cmake] 
[cmake]   IMPORTED_IMPLIB not set for imported target "QGpgme" configuration "Debug".
[cmake] This warning is for project developers.  Use -Wno-dev to suppress it.
[cmake] 
[cmake] -- Generating done (0.1s)
[cmake] -- Build files have been written to: C:/Users/Amalia/Desktop/Rachel/gpgme/build

This is patently clear in the contents of QGpgmeConfig.cmake lines 69 onwards:

# Create imported target QGpgme
add_library(QGpgme SHARED IMPORTED)

set_target_properties(QGpgme PROPERTIES
  IMPORTED_IMPLIB_RELEASE "${_IMPORT_PREFIX}/lib/libqgpgme.dll.a"
  INTERFACE_INCLUDE_DIRECTORIES "${_IMPORT_PREFIX}/include/qgpgme;${_IMPORT_PREFIX}/include"
  INTERFACE_LINK_LIBRARIES "Gpgmepp;Qt5::Core"
  IMPORTED_LOCATION "${_IMPORT_PREFIX}/bin/libqgpgme-15.dll"
)
get_target_property(_libpath QGpgme IMPORTED_IMPLIB_RELEASE)

And reviewing the patch that MSYS2 ships should've detected this bug on the spot (as it's a system artifact, the highlighted lines should not read RELEASE):

MINGW-packages/mingw-w64-gpgme/relocatable-cmake.patch

Lines 59 to 62 in 8817853

	- IMPORTED_IMPLIB_RELEASE "@resolved_libdir@/libqgpgme.dll.a"
	- INTERFACE_INCLUDE_DIRECTORIES "@resolved_includedir@/qgpgme;@resolved_includedir@"
	+ IMPORTED_IMPLIB_RELEASE "${_IMPORT_PREFIX}/lib/libqgpgme.dll.a"
	+ INTERFACE_INCLUDE_DIRECTORIES "${_IMPORT_PREFIX}/include/qgpgme;${_IMPORT_PREFIX}/include"

[Biswa96]

Thanks, I can reproduce the issue which should be fixed in upstream. IMPORTED_IMPLIB_RELEASE comes from upstream files

• https://git.gnupg.org/cgi-bin/gitweb.cgi?p=gpgmeqt.git;a=blob;f=src/QGpgmeQt6Config-w32.cmake.in.in
• https://git.gnupg.org/cgi-bin/gitweb.cgi?p=gpgmeqt.git;a=blob;f=src/QGpgmeConfig-w32.cmake.in.in

Meanwhile, if you have any patch to fix the warning please create a pull request.

[MehdiChinoune]

I think the issue is fixed now.