From a8be6e94e79f602d543a15afd44ef60e378b138f Mon Sep 17 00:00:00 2001 From: Laurent Pinchart Date: Fri, 13 Sep 2019 20:59:30 +0300 Subject: qcam: Fix compilation errors with clang-10 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit clang-10 has introduced the same deprecated-copy warning that appeared in gcc-9 and caused build issues with Qt header files. However, the clang version seems more sensitive, and detects issues that are not fixed in Qt 5.13, unlike gcc-9. Extend the logic that disables the warning for gcc-9 and Qt < 5.13 to cover clang-10 and all Qt versions. Signed-off-by: Laurent Pinchart Reviewed-by: Niklas Söderlund Reviewed-by: Kieran Bingham --- src/qcam/meson.build | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) (limited to 'src/qcam') diff --git a/src/qcam/meson.build b/src/qcam/meson.build index 5b877a84..5150631b 100644 --- a/src/qcam/meson.build +++ b/src/qcam/meson.build @@ -25,13 +25,12 @@ if qt5_dep.found() qt5_cpp_args = [ '-DQT_NO_KEYWORDS' ] # gcc 9 introduced a deprecated-copy warning that is triggered by Qt until - # Qt 5.13. Disable it manually. - if cc.get_id() == 'gcc' - gcc_version = cc.version().split('.') - qt5_version = qt5_dep.version().split('.') - if qt5_version[1].to_int() < 13 and gcc_version[0].to_int() >= 9 - qt5_cpp_args += [ '-Wno-deprecated-copy' ] - endif + # Qt 5.13. clang 10 introduced the same warning, but detects more issues + # that are not fixed in Qt yet. Disable the warning manually in both cases. + if ((cc.get_id() == 'gcc' and cc.version().version_compare('>=9.0') and + qt5_dep.version().version_compare('<5.13')) or + (cc.get_id() == 'clang' and cc.version().version_compare('>=10.0'))) + qt5_cpp_args += [ '-Wno-deprecated-copy' ] endif resources = qt5.preprocess(moc_headers: qcam_moc_headers, -- cgit v1.2.1