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

add support for Linux arm64 and Linux arm #1633

Merged
merged 1 commit into from
Nov 6, 2024

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
Collaborator

@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.

.github/workflows/release.yml Outdated Show resolved Hide resolved
.github/workflows/release.yml Outdated Show resolved Hide resolved
src/linux/Packaging.Linux/build.sh Outdated Show resolved Hide resolved
src/linux/Packaging.Linux/build.sh Outdated Show resolved Hide resolved
src/linux/Packaging.Linux/pack.sh Outdated Show resolved Hide resolved
src/linux/Packaging.Linux/pack.sh Outdated Show resolved Hide resolved
Co-Authored-By: Matthew John Cheetham <[email protected]>
@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
Collaborator

@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
Collaborator

@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.

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.

Make it easier to build for ARM
2 participants