Skip to content

Conversation

@qiangxuhui
Copy link
Contributor

Go-1.19 has added support for loong64 arch, this PR adds build support for loong64 arch.

Building

go mod tidy && make all

Testing

[qiangxuhui@chenguoqi-fedora-24 ~]$ ./git-lfs-linux-loong64 --version
git-lfs/3.4.0 (GitHub; linux loong64; go 1.20.12; git dc564aac)

@qiangxuhui qiangxuhui requested a review from a team as a code owner January 2, 2024 11:13
Copy link
Member

@bk2204 bk2204 left a comment

Choose a reason for hiding this comment

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

Mostly this looks like it's heading in the right direction, but there are a few changes we need to add on here.

script/upload's categorize_arch needs to learn about this (it should probably be “64-bit LoongArch”), as does docs/howto/release-git-lfs.md. In general, everywhere you see ppc64le, a similar change should be provided as well.

@qiangxuhui qiangxuhui force-pushed the main branch 2 times, most recently from 86fee76 to d61b3ea Compare January 4, 2024 02:24
@qiangxuhui
Copy link
Contributor Author

@bk2204 I made some modifications based on your comments.

Copy link
Member

@bk2204 bk2204 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 the patch! This looks good, and I'll merge it.

@bk2204 bk2204 merged commit 2db011f into git-lfs:main Jan 4, 2024
chrisd8088 added a commit to chrisd8088/git-lfs that referenced this pull request Nov 23, 2024
In commit d61b3ea of PR git-lfs#5607 we added
support in our Makefile and scripts to build and upload a version of our
Linux release assets for the LoongArch64 platform, and at the same time
we added an entry for this platform to the end of the list of Linux assets
in our release process documentation.

Before updating the documentation of our release process, we first reorder
this list to keep it in alphanumeric order, so as to match how assets are
listed in our public release notes.
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.

2 participants