~bzoltan/ubuntu-ui-toolkit/restructure_i18n

Viewing all changes in revision 2119.

  • Committer: Tarmac
  • Author(s): Loïc Molinari
  • Date: 2016-09-20 12:52:18 UTC
  • mfrom: (2090.1.15 uitk-test)
  • Revision ID: tarmac-20160920125218-jgolt52iwpfd4qa7
Cleaned up qmake Ubuntu packaging variables.

The qtbase packaging for Ubuntu exports the CFLAGS, CXXFLAGS and LFLAGS variables in the environment before running configure so that the qtbase shared libraries can be compiled with the required distribution defined compiler flags. That is fine (though I guess that could be handled better at the qtbase packaging level with maybe a dedicated prf) but it results in qt_build_config.prf defining the QMAKE_CFLAGS, QMAKE_CXXFLAGS and QMAKE_LFLAGS variables with these distribution specific flags. Since we load this prf, these variables end up being defined when building the toolkit for development purpose. This is problematic because -O2 and -g are always defined (whatever the debug or release config chosen by the developer), also these are GCC specific flags which makes our build break with other compilers. This is also ugly because preprocessor defines are directly set in the QMAKE_C*FLAGS. In order to workaround these issues, we unset these variables and set the different flags apart from -g and -O2 in the right qmake variables. This allows to keep compiling with the Debian flags but in a more flexible way. For the Debian packaging, we keep the former behaviour by simply passing the debian_build config.

Approved by ubuntu-sdk-build-bot, Benjamin Zeller, Zsombor Egri.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: