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

CI - Fix Building(CMake) on Windows 2019 #3856

Closed

Conversation

volovikariel
Copy link
Contributor

Motivation: I don't want CI builds to fail. This PR addresses the Windows 2019 Building(CMake) failures.

Problem analysis: Windows 2019 Building(CMake) runs are failing for us: https://github.com/flameshot-org/flameshot/actions/runs/13554611383/job/37886033174

Image

The failures occur in the QT Install step:

- name: Install Qt
uses: jurplel/install-qt-action@v2
with:
version: 5.15.2
target: desktop
dir: '${{ github.workspace }}/build/'

I spent some time looking into it, and it looks like jurplel/install-qt-action is was also having issues with python version 3.13. See this commit, which limits their python versions to 3.12 now: jurplel/install-qt-action@fc214cc

Image

Proposed solution:
Going from jurplel/install-qt-action@v2 to jurplel/install-qt-action@v4 should fix the issue (as their commit was made to v4).

My tests:
This change seems to make the build pass: https://github.com/volovikariel/flameshot/actions/runs/13568251686/job/37926340193?pr=7
image

Windows 2019 Building(CMake) runs are failing for us: https://github.com/flameshot-org/flameshot/actions/runs/13554611383/job/37886033174

![Image](https://github.com/user-attachments/assets/2c5c1b2a-2259-4064-94a3-20e227bcd765)

The failures occur in the QT Install step: https://github.com/flameshot-org/flameshot/blob/aad3a5e42e2e98af7bfb6bbda0e3d1b9c0be7497/.github/workflows/build_cmake.yml#L109-L114

I spent some time looking into it, and it looks like `jurplel/install-qt-action` is avoiding `3.13` runs because it fails for them too. See jurplel/install-qt-action@fc214cc

![Image](https://github.com/user-attachments/assets/22b6926f-fa8f-461b-8b97-5b77e92f33d0)

Perhaps we just need to update our `install-qt-action` version
Remove QEMU dummy package installation from Linux-pack.yml (flameshot-org#3851)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant