-
Notifications
You must be signed in to change notification settings - Fork 1.3k
harfbuzz (and deps): enable for clangarm64 #10377
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
Conversation
makedepends=("${MINGW_PACKAGE_PREFIX}-python-setuptools" | ||
"${MINGW_PACKAGE_PREFIX}-python-setuptools-scm") | ||
#checkdepends=("${MINGW_PACKAGE_PREFIX}-python-pytest-black" | ||
checkdepends=("${MINGW_PACKAGE_PREFIX}-python-pytest-black") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one is actually needed for the checks to pass. Have added python-pytest-black
for arm64 as well 👍🏼
https://github.com/msys2-arm/MINGW-packages/runs/4589639443?check_suite_focus=true
No idea what that is supposed to mean. |
@jeremyd2019 that's weird. The problematic UWP package ID seems to belong to the "UWP File Explorer": https://www.thurrott.com/windows/windows-10/115275/windows-10-tip-unlock-uwp-file-explorer What Windows arm64 build are you running? I'm on 22000.376. Maybe there was some Windows Store update in progress while you attempted the build? It works correctly on my machine: gobject-introspection.txt Also, reinstalling that specific app might help: https://stackoverflow.com/a/67715630/4337765 |
The runner was 22000.376, but I expect my scripts to automate setting up a runner from a fresh Windows are to blame. I've seen that it doesn't seem to set up the "modern" apps entirely (it may be the responsibility of some background job after first logon?), and also I set the default shell to a CMD script that launches the runner, so explorer isn't actually running at the time the jobs are run. |
It succeeds on my raspberry pi (which also isn't running explorer), so it's probably a matter of the UWP apps not being deployed into the user profile entirely. |
Yep, need a new command added to my scripts when setting up a new Windows instance. Unfortunately, what I found uses powershell. That's pretty slow, especially under TCG. I already can't really use TCG for much because it's too slow, but I sometimes test my scripts on it. |
For the record, this was: <CommandLine>powershell -ExecutionPolicy Bypass -Command "Get-AppXPackage | Foreach {Add-AppxPackage -DisableDevelopmentMode -Register """"$($_.InstallLocation)\AppXManifest.xml""""}"</CommandLine> The quadruple-double quotes were great. |
As requested in #10333 (comment)
I need
harfbuzz
forpango
, but since it has quite a large dependency tree, I figured it'd be better to create a separate PR for this package first.I was able to build all packages on my arm64 device, but the checks for
python-regex
fail with:That error seems unrelated to arm64 specifically. I ran the build with
--nocheck
for now.