Details
-
Bug
-
Resolution: Done
-
P2: Important
-
None
-
5.6.0 RC
-
None
Description
See: http://testresults.qt.io/coin/integration/qt/qtbase/tasks/1457090702.thrift_bin
Please remove OpenSSL from INCLUDE/LIB for the WinCE builds. Only set them for desktop builds.
The Windows CI virtual machine template do add C:\openssl\include and c:\openssl\lib to the INCLUDE and LIB environment variables. This is a problem for cross-compilation builds and it should only be done when not cross compiling. This can be fixed in three steps:
(1) In the WindowsTargetEnvironment class in targetenvironments.py in the prepareEnvironment function we should add c:\openssl\include to INCLUDE and c:\openssl\lib to LIB. This also requires updating the test data using test_platformconfigurations.py --generate.
(2) That change should be taken into production.
(3) Then the windows virtual machine templates should be modified to not automatically add those paths to INCLUDE and LIB, through the control panel.
Then the last step is completed and in production, then https://codereview.qt-project.org/#/c/151207/ can be integrated into Qt.
Attachments
For Gerrit Dashboard: QTQAINFRA-971 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
151412,3 | Adding openssl env vars for WIN | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |