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 missing Linux platform triples in autoconf tool. #5548

Merged
merged 4 commits into from
Sep 1, 2024

Conversation

Redbeanw44602
Copy link
Contributor

解决 #5530

第一次pr,有很多不懂的地方还望指点。😂

不知道 trybuild 里的写的对不对,package/tools 应该没有问题,测试过了。

@Redbeanw44602 Redbeanw44602 changed the title Add missing Linux platform triplet in autoconf tool. Add missing Linux platform triples in autoconf tool. Aug 30, 2024
elseif package:is_plat("linux") then
local triples =
{
["arm64-v8a"] = "aarch64-linux-gnu",
Copy link
Member

Choose a reason for hiding this comment

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

还有 arm64

Copy link
Contributor Author

@Redbeanw44602 Redbeanw44602 Aug 31, 2024

Choose a reason for hiding this comment

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

set_archs("i386", "x86_64", "armv7", "armv7s", "arm64-v8a", "mips", "mips64", "mipsel", "mips64el", "loong64")

linux 下面好像并没有叫 arm64 的处理器架构,只有 arm64-v8a。

mips64el = "mips64el-linux-gnu",
loong64 = "loongarch64-linux-gnu"
}
table.insert(configs, "--host=" .. (triples[package:arch()] or triples.i386))
Copy link
Member

Choose a reason for hiding this comment

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

现在 i386 很少用了,默认改成 x86_64

Copy link
Contributor Author

Choose a reason for hiding this comment

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

看上面写的都是 i386 所以就写 i386 了,要不都改过来?

@waruqi waruqi added this to the v2.9.5 milestone Aug 31, 2024
@waruqi waruqi merged commit 0d2bd15 into xmake-io:dev Sep 1, 2024
20 checks passed
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