Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-115045

Don't overwrite my CMAKE_<Config>_POSTFIX

    XMLWordPrintable

Details

    • All
    • 8c4112511 (dev), 3ec24c695 (6.6), 01718db0e (tqtc/lts-6.5)

    Description

      I set CMAKE_RELWITHDEBINFO_POSTFIX=rd but the build system changes it to CMAKE_RELWITHDEBINFO_POSTFIX=_relwithdebinfo durring configuration.

      CMAKE_DEBUG_POSTFIX=d seams to work as expected. (I didn't test if you overwrite it internaly with d as well.) CMAKE_DEBUG_POSTFIX is overwritten with d as well.

      This patch fixed it for my use case:

      diff --git a/qt_original/qtbase/cmake/QtSetup.cmake b/qt/qtbase/cmake/QtSetup.cmake
      index 5a37320..d9d6b51 100644
      --- a/qt_original/qtbase/cmake/QtSetup.cmake
      +++ b/qt/qtbase/cmake/QtSetup.cmake
      @@ -90,7 +90,9 @@ if(QT_GENERATOR_IS_MULTI_CONFIG AND CMAKE_CONFIGURATION_TYPES)
      
               string(TOLOWER "${__qt_setup_config_type}" __qt_setup_config_type_lower)
               string(TOUPPER "${__qt_setup_config_type}" __qt_setup_config_type_upper)
      -        set(CMAKE_${__qt_setup_config_type_upper}_POSTFIX "_${__qt_setup_config_type_lower}")
      +        if("${CMAKE_${__qt_setup_config_type_upper}_POSTFIX}" STREQUAL "")
      +            set(CMAKE_${__qt_setup_config_type_upper}_POSTFIX "_${__qt_setup_config_type_lower}")
      +        endif()
               if(APPLE)
                   set(CMAKE_FRAMEWORK_MULTI_CONFIG_POSTFIX_${__qt_setup_config_type_upper}
                       "_${__qt_setup_config_type_lower}")
      

      Note that this is not a general patch! It probably doesn't work CMAKE_DEBUG_POSTFIX because that is changed later in the file. Also on Apple systems there seams to be some Framework stuff that is not taken into account. **

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          People

            semlanik Alexey Edelev
            bebuch Benjamin Buch
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes