-
Notifications
You must be signed in to change notification settings - Fork 167
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
gdk-pixbuf fails to compile in CI #1436
Comments
2024.8.1 works fine, this is a regression in 2024.9.0 |
2024.10.0 is still broken the same way, here is CI run: https://github.com/autonomys/space-acres/actions/runs/11266017698/job/31328812842 |
Probably related to #1378 |
Hi @nazar-pc, would you like to create a minimum producible CI workflow so that we can try to reproduce this? The main differences that I see compared to our CI workflow that does pass is:
|
The workflow is as simple as it gets: name: Release
on:
workflow_dispatch:
jobs:
release:
runs-on: windows-2022
steps:
- name: Install GTK4 (Windows)
run: |
pipx install gvsbuild
gvsbuild build gtk4 librsvg Here is the result of it: https://github.com/autonomys/space-acres/actions/runs/11284587468/job/31386012839 |
Hi @nazar-pc, I am able to reproduce this. It is because of the space in the path to pipx. An alternative that works is:
This is either an upstream issue with Meson or Ninja. I'll try to narrow it down and get it fixed. |
Yeah, spaces on Windows are a special kind of pain: https://github.com/autonomys/subspace/blob/1755b51f545b27be006e5f10ccfc4123b0efbb5b/.github/workflows/snapshot-build.yml#L219-L220 Thanks for looking into it! |
Looks like the latest version was pulled in my project in CI and it now doesn't compile:
Looking at python path, it doesn't seem to be handling white spaces correctly.
CI logs: https://github.com/autonomys/space-acres/actions/runs/10834949725/job/30065354680?pr=294#step:6:16918
The text was updated successfully, but these errors were encountered: