Coin: remove workarounds to allow running Android tests

We had two workarounds:
* script that adds Gui to tests
* create a symbolic link for the qt install dir to fake_prefix which
androiddelployqt was expecting them to be under

Both issues are fixed, thus removing the workarounds.

Pick-to: 6.1 6.0
Change-Id: Ic022bece15afe92c693d573893d260b13b4227ed
Reviewed-by: Heikki Halmet <heikki.halmet@qt.io>
This commit is contained in:
Assam Boudjelthia 2021-02-18 13:29:00 +02:00
parent a2e23bca0f
commit bcbdbd50fe
4 changed files with 0 additions and 60 deletions

View File

@ -162,6 +162,4 @@ function(qt_internal_install_android_helper_scripts)
qt_path_join(destination "${QT_INSTALL_DIR}" "${INSTALL_LIBEXECDIR}") qt_path_join(destination "${QT_INSTALL_DIR}" "${INSTALL_LIBEXECDIR}")
qt_copy_or_install(PROGRAMS "${CMAKE_CURRENT_SOURCE_DIR}/util/android/android_emulator_launcher.sh" qt_copy_or_install(PROGRAMS "${CMAKE_CURRENT_SOURCE_DIR}/util/android/android_emulator_launcher.sh"
DESTINATION "${destination}") DESTINATION "${destination}")
qt_copy_or_install(PROGRAMS "${CMAKE_CURRENT_SOURCE_DIR}/util/android/android_cmakelist_patcher.sh"
DESTINATION "${destination}")
endfunction() endfunction()

View File

@ -1,17 +1,5 @@
type: Group type: Group
instructions: instructions:
- type: ChangeDirectory
directory: "{{.SourceDir}}"
- type: ExecuteCommand
command: "{{.InstallDir}}/libexec/android_cmakelist_patcher.sh"
maxTimeInSeconds: 6000
maxTimeBetweenOutput: 1200
userMessageOnFailure: >
Failed to patch CMakeLists.txt.
enable_if:
condition: property
property: features
contains_value: AndroidTestRun
- type: ChangeDirectory - type: ChangeDirectory
directory: "{{.BuildDir}}" directory: "{{.BuildDir}}"
- type: ExecuteCommand - type: ExecuteCommand

View File

@ -11,13 +11,3 @@ instructions:
Failed to install tests archive. Failed to install tests archive.
- type: ChangeDirectory - type: ChangeDirectory
directory: "{{.SourceDir}}_standalone_tests" directory: "{{.SourceDir}}_standalone_tests"
- type: ExecuteCommand
command: "ln -s {{.InstallDir}}/target fake_prefix"
maxTimeInSeconds: 6000
maxTimeBetweenOutput: 1200
userMessageOnFailure: >
Failed to run android hack. See QTBUG-88579 to check if this can be removed
enable_if:
condition: property
property: features
contains_value: AndroidTestRun

View File

@ -1,36 +0,0 @@
#!/bin/sh
#############################################################################
##
## Copyright (C) 2020 The Qt Company Ltd.
## Contact: https://www.qt.io/licensing/
##
## This file is part of the plugins of the Qt Toolkit.
##
## $QT_BEGIN_LICENSE:GPL-EXCEPT$
## Commercial License Usage
## Licensees holding valid commercial Qt licenses may use this file in
## accordance with the commercial license agreement provided with the
## Software or, alternatively, in accordance with the terms contained in
## a written agreement between you and The Qt Company. For licensing terms
## and conditions see https://www.qt.io/terms-conditions. For further
## information use the contact form at https://www.qt.io/contact-us.
##
## GNU General Public License Usage
## Alternatively, this file may be used under the terms of the GNU
## General Public License version 3 as published by the Free Software
## Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
## included in the packaging of this file. Please review the following
## information to ensure the GNU General Public License requirements will
## be met: https://www.gnu.org/licenses/gpl-3.0.html.
##
## $QT_END_LICENSE$
##
#############################################################################
# This util patches CMakeLists.txt files to enable running
# autotest in Android emulator in Qt CI.
tests_dir="$PWD/tests/auto"
find ${tests_dir} -iname "CMakeLists.txt" -print0 |
while IFS= read -r -d '' file; do
sed -i '/qt_internal_add_test/a\ PUBLIC_LIBRARIES\n Qt::Gui' $file
done