Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: fix error when instaling win64_llvm_mingw / Qt 6.7 and 6.8 #826

Merged
merged 8 commits into from
Oct 12, 2024

Conversation

miurahr
Copy link
Owner

@miurahr miurahr commented Oct 5, 2024

ref #825

@miurahr miurahr self-assigned this Oct 5, 2024
@miurahr miurahr added the bug Something isn't working label Oct 5, 2024
@miurahr miurahr force-pushed the topic/miurahr/fix-win64-llvm-mingw branch from 780b033 to e38fb3a Compare October 5, 2024 23:45
@miurahr
Copy link
Owner Author

miurahr commented Oct 6, 2024

Windows install-qt 6.7.3 win64_llvm_mingw for desktop failed as expected.

2024-10-05 23:50:37,697 - aqt.main - ERROR - installer 6316 Updater caused an IO error: [Errno 2] No such file or directory: 'D:\\a\\1\\b\\Qt\\6.7.3\\llvm_mingw\\mkspecs\\qconfig.pri'

@miurahr miurahr force-pushed the topic/miurahr/fix-win64-llvm-mingw branch from 349bfd9 to 88746ed Compare October 6, 2024 00:31
- set arch_dir to be llvm_mingw_64

Signed-off-by: Hiroshi Miura <[email protected]>
@miurahr miurahr force-pushed the topic/miurahr/fix-win64-llvm-mingw branch from 88746ed to 7cfd15a Compare October 6, 2024 00:52
@miurahr miurahr marked this pull request as ready for review October 6, 2024 00:52
@miurahr
Copy link
Owner Author

miurahr commented Oct 6, 2024

failed

2024-10-06 01:06:05,578 - aqt.main - ERROR - installer 1208 Updater caused an IO error: [Errno 2] No such file or directory: 'D:\\a\\1\\b\\Qt\\6.7.3\\llvm_mingw_64\\mkspecs\\qconfig.pri'

aqt/metadata.py Outdated Show resolved Hide resolved
@miurahr
Copy link
Owner Author

miurahr commented Oct 6, 2024

Patchings are worked as expected now.

2024-10-06 01:40:38,859 - aqt.updater - INFO - updater 4312 Patching D:\a\1\b\Qt\6.7.3\llvm-mingw_64\bin\qmake.exe
2024-10-06 01:40:38,859 - aqt.updater - INFO - updater 4312 Patching D:\a\1\b\Qt\6.7.3\llvm-mingw_64\lib\pkgconfig\Qt6Concurrent.pc
2024-10-06 01:40:38,859 - aqt.updater - INFO - updater 4312 Patching D:\a\1\b\Qt\6.7.3\llvm-mingw_64\lib\pkgconfig\Qt6Core.pc
2024-10-06 01:40:38,859 - aqt.updater - INFO - updater 4312 Patching D:\a\1\b\Qt\6.7.3\llvm-mingw_64\lib\pkgconfig\Qt6DBus.pc
2024-10-06 01:40:38,859 - aqt.updater - INFO - updater 4312 Patching D:\a\1\b\Qt\6.7.3\llvm-mingw_64\lib\pkgconfig\Qt6Designer.pc
2024-10-06 01:40:38,859 - aqt.updater - INFO - updater 4312 Patching D:\a\1\b\Qt\6.7.3\llvm-mingw_64\lib\pkgconfig\Qt6Gui.pc
2024-10-06 01:40:38,859 - aqt.updater - INFO - updater 4312 Patching D:\a\1\b\Qt\6.7.3\llvm-mingw_64\lib\pkgconfig\Qt6Help.pc
...
2024-10-06 01:40:38,905 - aqt.updater - INFO - updater 4312 Patching D:\a\1\b\Qt\6.7.3\llvm-mingw_64\lib\Qt6Widgets.prl
2024-10-06 01:40:38,905 - aqt.updater - INFO - updater 4312 Patching D:\a\1\b\Qt\6.7.3\llvm-mingw_64\lib\Qt6Xml.prl

but it is still failed on build test.

Starting: build test with qmake w/o extra module
==============================================================================
Task         : PowerShell
Description  : Run a PowerShell script on Linux, macOS, or Windows
Version      : 2.245.1
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/powershell
==============================================================================
Generating script.
========================== Starting Command Output ===========================
"C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'D:\a\_temp\39650071-c659-4534-b38c-e81e4ac5672f.ps1'"
INFO    : aqtinstall(aqt) v3.1.19.dev24 on Python 3.12.6 [CPython MSC v.1940 64 bit (AMD64)]
ERROR   : The package 'qt.tools.' was not found while parsing XML of package information!
==============================Suggested follow-up:==============================
* Please use 'aqt list-tool windows desktop tools_mingw' to show tool variants available.
Failed to install tools_mingw
At D:\a\_temp\39650071-c659-4534-b38c-e81e4ac5672f.ps1:25 char:5
+     throw 'Failed to install tools_mingw'
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Failed to install tools_mingw:String) [], RuntimeException
    + FullyQualifiedErrorId : Failed to install tools_mingw

@pcolby could you help to investigate it?

@miurahr miurahr added the help wanted Extra attention is needed label Oct 6, 2024
@miurahr
Copy link
Owner Author

miurahr commented Oct 6, 2024

 python -m aqt list-tool windows desktop tools_mingw
qt.tools.mingw47
qt.tools.win32_mingw48
qt.tools.win32_mingw482
qt.tools.win32_mingw491
qt.tools.win32_mingw492
qt.tools.win32_mingw530
qt.tools.win32_mingw730
qt.tools.win32_mingw810
qt.tools.win64_mingw730
qt.tools.win64_mingw810

No entry for

@miurahr
Copy link
Owner Author

miurahr commented Oct 6, 2024

@ddalcino
Copy link
Contributor

ddalcino commented Oct 6, 2024

No entry for

Looks all right to me.

$ python -m aqt list-tool windows desktop | grep mingw
tools_mingw1310
tools_mingw90
tools_mingw
tools_llvm_mingw1706

$ python -m aqt list-tool windows desktop tools_llvm_mingw1706
qt.tools.win64_llvm_mingw1706

$ python -m aqt list-tool windows desktop tools_mingw1310  
qt.tools.win64_mingw1310

Looks like:

aqtinstall/ci/steps.yml

Lines 291 to 292 in 7fbe587

python -m aqt install-tool $(if (($QT_BASE_MIRROR + "") -ne "") { "-b $QT_BASE_MIRROR" } else {""}) `
--outputdir $(Build.BinariesDirectory)/Qt $(HOST) desktop tools_mingw qt.tools.$(MINGW_VARIANT)

... needs to add logic for the alternate cases, when you need to install tools_llvm_mingw1706 or tools_mingw1310 instead of one of the older varieties of tools_mingw

@ddalcino
Copy link
Contributor

ddalcino commented Oct 6, 2024

#829 should make this work

@miurahr
Copy link
Owner Author

miurahr commented Oct 9, 2024

I've pulled @ddalcino changes to the topic branch. Thanks.

Ignore autodesktop for llvm-mingw case
@miurahr miurahr merged commit 0392095 into master Oct 12, 2024
49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants