From c0173d343d41bca94e8b1234821467903fa590a6 Mon Sep 17 00:00:00 2001 From: autoantwort <41973254+autoantwort@users.noreply.github.com> Date: Wed, 19 Apr 2023 03:51:50 +0200 Subject: [PATCH] [qt5compat] add proper error message (#30829) --- ports/qt5compat/portfile.cmake | 7 ++++++- ports/qt5compat/vcpkg.json | 1 + ports/qtbase/portfile.cmake | 2 ++ ports/qtbase/vcpkg.json | 2 +- versions/baseline.json | 4 ++-- versions/q-/qt5compat.json | 5 +++++ versions/q-/qtbase.json | 5 +++++ 7 files changed, 22 insertions(+), 4 deletions(-) diff --git a/ports/qt5compat/portfile.cmake b/ports/qt5compat/portfile.cmake index 452cf48be9..d13a078b94 100644 --- a/ports/qt5compat/portfile.cmake +++ b/ports/qt5compat/portfile.cmake @@ -19,7 +19,12 @@ INVERTED_FEATURES #For iconv feature to work the following must be true: #CONDITION NOT FEATURE_icu AND FEATURE_textcodec AND NOT WIN32 AND NOT QNX AND NOT ANDROID AND NOT APPLE AND WrapIconv_FOUND -#TODO: check if qtbase was built with ICU and fail if iconv is given here. +if("iconv" IN_LIST FEATURES) + include("${SCRIPT_PATH}/port_status.cmake") + if(qtbase_with_icu) + message(FATAL_ERROR "qtbase was built with ICU. The iconv feature is not compatible with ICU.") + endif() +endif() set(TOOL_NAMES) qt_install_submodule(PATCHES ${${PORT}_PATCHES} diff --git a/ports/qt5compat/vcpkg.json b/ports/qt5compat/vcpkg.json index 9d17a62bd0..6c5f53c8c2 100644 --- a/ports/qt5compat/vcpkg.json +++ b/ports/qt5compat/vcpkg.json @@ -1,6 +1,7 @@ { "name": "qt5compat", "version": "6.4.3", + "port-version": 1, "description": "The module contains unsupported Qt 5 APIs", "homepage": "https://www.qt.io/", "license": null, diff --git a/ports/qtbase/portfile.cmake b/ports/qtbase/portfile.cmake index 718755cfb7..a2b6e4e626 100644 --- a/ports/qtbase/portfile.cmake +++ b/ports/qtbase/portfile.cmake @@ -309,6 +309,8 @@ file(COPY "${CURRENT_PACKAGES_DIR}/share/${PORT}" ) +file(CONFIGURE OUTPUT "${CURRENT_PACKAGES_DIR}/share/${PORT}/port_status.cmake" CONTENT "set(qtbase_with_icu ${FEATURE_icu})\n") + set(other_files qt-cmake qt-cmake-private qt-cmake-standalone-test diff --git a/ports/qtbase/vcpkg.json b/ports/qtbase/vcpkg.json index be16a2a56b..f3b562bd35 100644 --- a/ports/qtbase/vcpkg.json +++ b/ports/qtbase/vcpkg.json @@ -1,7 +1,7 @@ { "name": "qtbase", "version": "6.4.3", - "port-version": 2, + "port-version": 3, "description": "Qt Application Framework Base Module. Includes Core, GUI, Widgets, Networking, SQL, Concurrent and other essential qt components.", "homepage": "https://www.qt.io/", "license": null, diff --git a/versions/baseline.json b/versions/baseline.json index 82328c583b..faea367488 100644 --- a/versions/baseline.json +++ b/versions/baseline.json @@ -6594,7 +6594,7 @@ }, "qt5compat": { "baseline": "6.4.3", - "port-version": 0 + "port-version": 1 }, "qt6betablock": { "baseline": "6.2.0-beta", @@ -6610,7 +6610,7 @@ }, "qtbase": { "baseline": "6.4.3", - "port-version": 2 + "port-version": 3 }, "qtcharts": { "baseline": "6.4.3", diff --git a/versions/q-/qt5compat.json b/versions/q-/qt5compat.json index d0821b58b7..3d9f3174f4 100644 --- a/versions/q-/qt5compat.json +++ b/versions/q-/qt5compat.json @@ -1,5 +1,10 @@ { "versions": [ + { + "git-tree": "0a79c6cfb2113f764e603ee13a4c12493f16b6d5", + "version": "6.4.3", + "port-version": 1 + }, { "git-tree": "df64fa51cb5aaaf59cbe76b91476db4b5c272152", "version": "6.4.3", diff --git a/versions/q-/qtbase.json b/versions/q-/qtbase.json index c3cb70e889..d7808b0279 100644 --- a/versions/q-/qtbase.json +++ b/versions/q-/qtbase.json @@ -1,5 +1,10 @@ { "versions": [ + { + "git-tree": "cc1cc5c1a5d6a7c4c10feebe8d43a069d3a375a7", + "version": "6.4.3", + "port-version": 3 + }, { "git-tree": "b694b6e51c89c8610540165e6e8b3a236acb4301", "version": "6.4.3",