Skip to content

github: Drop arch-mingw-github - #389

Merged
jp7677 merged 1 commit into
masterfrom
jp7677-patch-1
Aug 28, 2026
Merged

jp7677 merged 1 commit into
masterfrom
jp7677-patch-1

Conversation

@jp7677

@jp7677 jp7677 commented Aug 23, 2026

Copy link
Copy Markdown
Owner

This pulls in less packages, thus faster, and avoids AUR.

@Saancreed Saancreed left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, so 👍 from me.

Or, we could just drop the action and install the deps ourselves. All we need is archlinux:base-devel and pacman -S git meson mingw-w64-gcc ninja. Plus maybe clang-format for linting and wine for running tests. This is how I do this in wine-nvml, I'd consider it somewhat simpler than our current setup.

@WinterSnowfall

WinterSnowfall commented Aug 25, 2026 •

Copy link
Copy Markdown

Or, we could just drop the action and install the deps ourselves. All we need is archlinux:base-devel and pacman -S git meson mingw-w64-gcc ninja. Plus maybe clang-format for linting and wine for running tests. This is how I do this in wine-nvml, I'd consider it somewhat simpler than our current setup.

Probably best to just do this IMHO. I've only created that fork as a stop gap measure and can't guarantee it will be around for all eternity. dxvk/vkd3d-proton will most likely gravitate towards the same approach of installing the deps directly on the base image in the near future.

@jp7677
jp7677 force-pushed the jp7677-patch-1 branch 2 times, most recently from 49148be to a910405 Compare August 28, 2026 10:42
@jp7677 jp7677 changed the title github: Switch to arch-mingw-github fork github: Drop arch-mingw-github Aug 28, 2026
@jp7677
jp7677 force-pushed the jp7677-patch-1 branch 3 times, most recently from c30165f to 3e97da7 Compare August 28, 2026 10:49
@jp7677
jp7677 marked this pull request as ready for review August 28, 2026 10:56
Install build tooling ourselves. This pulls in less packages,
thus faster, and still avoids AUR.
@jp7677
jp7677 merged commit d102231 into master Aug 28, 2026
4 checks passed
@jp7677
jp7677 deleted the jp7677-patch-1 branch September 5, 2026 12:14
@WinterSnowfall

WinterSnowfall commented Sep 16, 2026 •

Copy link
Copy Markdown

FYI, not sure whether you are using git anywhere within the CI flows, but if you are it's probably regressed now and needs adding the GitHub workspace as a safe directory: doitsujin/dxvk#5905

@jp7677

jp7677 commented Sep 19, 2026

Copy link
Copy Markdown
Owner Author

FYI, not sure you are using git anywhere within the CI flows, but if you are it's probably regressed now and needs adding the GitHub workspace as a safe directory: doitsujin/dxvk#5905

Thanks a lot for the heads up, addressed in #397 with an additional step that fails when git wont work with us. So no more silent drops.

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.

3 participants