Skip to content

mingw-w64-git-credential-manager: don't cross-polute archs#673

Merged
dscho merged 1 commit intomainfrom
gcm-build-fix
Feb 3, 2026
Merged

mingw-w64-git-credential-manager: don't cross-polute archs#673
dscho merged 1 commit intomainfrom
gcm-build-fix

Conversation

@mjcheetham
Copy link
Member

We have been downloading and extracting the pre-built binaries for Git Credential Manager to the same $srcdir directory, meaning that when we build multiple architectures we're not always starting from a clean slate and have been grabbing binaries from other architectures.

Before we added support for x86_64 and clangarm64 native binaries, this was OK since we only downloaded the i686 flavour of GCM files.

Update the PKGBUILD to no longer extract the GCM binaries, but manually extract them to an architecture-specific directory - this way we don't cross-contaminate.

@mjcheetham mjcheetham requested a review from dscho January 22, 2026 17:25
We have been downloading and extracting the pre-built binaries for Git
Credential Manager to the same $srcdir directory, meaning that when we
build multiple architectures we're not always starting from a clean
slate and have been grabbing binaries from other architectures.

Before we added support for x86_64 and clangarm64 native binaries, this
was OK since we only downloaded the i686 flavour of GCM files.

Update the PKGBUILD to no longer extract the GCM binaries, but manually
extract them to an architecture-specific directory - this way we don't
cross-contaminate.

Signed-off-by: Matthew John Cheetham <mjcheetham@outlook.com>
Copy link
Member

@dscho dscho left a comment

Choose a reason for hiding this comment

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

Thanks, and sorry for the delay in merging!

@dscho dscho merged commit 70e451a into main Feb 3, 2026
7 checks passed
@dscho dscho deleted the gcm-build-fix branch February 3, 2026 09:44
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