-
Notifications
You must be signed in to change notification settings - Fork 346
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 dependencies that support the loongarch64 architecture #2799
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: yzewei <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your first contribution! Let me know why this change leads to support the loongarch64 arch?
Thanks for your reply! |
@yzewei I understood, but we don't have the CI platform to test the loongarch64 arch. It means, unfortunately, it's hard to maintain it for us. Do you have any good ideas? |
@utam0k |
@yzewei Thanks for your response. Who will maintain the loongarch64 arch? In other words, is it worth the maintenance? Would you use it? Sorry for the spicy question, but we consider it because our bandwidth is very limited. |
@utam0k It doesn't matter, the reason why I submitted pr is because the alpine community is about to support loongacrh. As an open-source contributor to the loongacrh architecture, I found on aports that youki has not updated libc, resulting in the failure of the package compilation under loongarch. If you add loongarch support, I believe that with the active number of people in the alpine community, there will be a lot of people using it |
Why so? Who will continue to maintain the loongarch64 arch in youki? Do you want to contribute youki to supporting it?
Do you run containers on Alpine? Also, we need to review our dependencies. For example opencontainers/runtime-spec#1187 |
I share the same concern. We have limited bandwidth to support an extra architecture at the moment. With that being said, I am unclear how dropping the explicit |
Add dependencies that support the loongarch64 architecture
rust-lang/libc#2765