Skip to content

Conversation

theofficialgman
Copy link
Contributor

@theofficialgman theofficialgman commented Jun 5, 2024

Adds support for building and publishing Linux ARM64 and Linux ARM32

replaces #237
replaces #1526
closes #606

needed for GitHub Desktop which uses dugite desktop/desktop#18700 desktop/dugite-native#510

CC @niik @shiftkey

continuous-integration works https://github.com/theofficialgman/git-credential-manager/actions/runs/9391634166
can't test release due to lack of keys

@mjcheetham mjcheetham self-assigned this Jun 5, 2024
@theofficialgman
Copy link
Contributor Author

@shiftkey needed to support GitHub Desktop linux on all currently built architectures. Please help this get merged.

@theofficialgman
Copy link
Contributor Author

bump
@niik @shiftkey please help this dependent PR for desktop/desktop#18700 get merged

Copy link
Contributor

@mjcheetham mjcheetham left a comment

Choose a reason for hiding this comment

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

Thanks for this! Adding Linux ARM support has been a long standing request, but we've just not had time to get around to it, or time to review other submissions.

I think your changes here work good! The only request I have is let's take this time to change the gcm-linux_$ARCH artifact format to match the others: gcm-$RUNTIME.

This way we also only pass the same $RUNTIME argument to all the build scripts, and only compute the $ARCH for the Debian control file in pack.sh.

Co-Authored-By: Matthew John Cheetham <mjcheetham@github.com>
@theofficialgman
Copy link
Contributor Author

@mjcheetham I implemented the changes, squashed (since there was really no need to keep the history, it just would make it more confusing), and gave you co-authorship.

The changes should be finished.

Copy link
Contributor

@mjcheetham mjcheetham left a comment

Choose a reason for hiding this comment

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

This is great work! Thank you

@mjcheetham mjcheetham merged commit 31ce754 into git-ecosystem:main Nov 6, 2024
10 checks passed
@theofficialgman
Copy link
Contributor Author

@mjcheetham https://github.com/git-ecosystem/git-credential-manager/actions/runs/11706008472/job/32602083045#step:5:344

/__w/git-credential-manager/git-credential-manager/src/linux/Packaging.Linux/build.sh: line 54: dpkg-architecture: command not found 

that is an oversight. I was not aware this scripting was used in anything other than the debian based release actions.

@mjcheetham
Copy link
Contributor

@mjcheetham https://github.com/git-ecosystem/git-credential-manager/actions/runs/11706008472/job/32602083045#step:5:344

/__w/git-credential-manager/git-credential-manager/src/linux/Packaging.Linux/build.sh: line 54: dpkg-architecture: command not found 

that is an oversight. I was not aware this scripting was used in anything other than the debian based release actions.

Thanks for alerting me. I am looking at this now.

@theofficialgman
Copy link
Contributor Author

@mjcheetham now that this has been included for a few weeks, is there any timeline for a new release to be made? I know there are some dependent projects waiting on this.

@natescherer
Copy link

+1 on getting a new release out that has arm64 debs. It's currently a big pain to try and build a dual amd64/arm64 container as you need the overhead of installing the .NET SDK and then installing it as a tool.

@theofficialgman
Copy link
Contributor Author

@mjcheetham just checking in on this and noticed that this is still not in the latest release https://github.com/git-ecosystem/git-credential-manager/releases/tag/v2.6.1 . Any timeline on when a tag will be made on main head?

@dscho
Copy link
Contributor

dscho commented Sep 19, 2025

@theofficialgman I have no official timeline, even though I am happy to report that the ownership issues seem to have been resolved now. Hopefully it is now only a matter of weeks at most until a new release can be made.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

8 participants