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

build: update workflow and change to the new ARM64 runner #168

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

xujiajiadexiaokeai
Copy link

感谢大佬提供的分词库🙏,最近在做的一个小工具需要做全文搜索,看到了大佬的工作,非常好用。

之前也遇到了和 #154 相同的问题,近日发现 ubuntu-22.04-arm 已上线。

本次修改移除了当前 workflow 中使用 QEMU 来模拟构建 aarch64 版本 libsimple 的job,在原有 Linux 构建 job 中新增了 ubuntu-22.04-arm 。

本次修改后, ARM 版本 libsimple 的 release 包名将更新为 libsimple-linux-ubuntu-22.04-arm.zip ,也将与其他版本的命名格式保持一致。

本次修改已在个人仓库中经过测试,详情见 https://github.com/xujiajiadexiaokeai/simple/actions/runs/12887702219

如有其他问题,请联系我,谢谢。

@wangfenjin
Copy link
Owner

非常感谢!

@xujiajiadexiaokeai
Copy link
Author

@wangfenjin 您好,目前ci过程的报错我昨天也遇到过,是因为我们使用的 ubuntu-latest 已经从22.04版本升级到了24.04版本,在最新的版本中, lcov 也开始使用2.x版本,而不是之前的1.x版本,导致了目前的错误。

因为我不太熟悉 lcov ,昨天也是参考了 Copilot 的解决方法,添加了 --ignore-errors 选项后,解决了报错的问题:
xujiajiadexiaokeai@8fe8927#diff-7829468e86c1cc5d5133195b5cb48e1ff6c75e3e9203777f6b2e379d9e4882b3R192
xujiajiadexiaokeai@8fe8927#diff-7829468e86c1cc5d5133195b5cb48e1ff6c75e3e9203777f6b2e379d9e4882b3R195
但是我不确定是否应该忽略这个报错,就没有提交这部分commit,如果需要的话我可以补上来。

第二种解决方案是暂时不使用 ubuntu-latest ,先修改为 ubuntu-22.04 ,保持和原来的runner版本一致,之后再处理报错。

@wangfenjin
Copy link
Owner

对,我也发现了

--ignore-errors 后能正常算覆盖率吗?如果可以的话你再帮忙做个提交吧

@wangfenjin
Copy link
Owner

我还在考虑升级 run-cmake 那些东西,但是最近比较忙

@xujiajiadexiaokeai
Copy link
Author

可以计算覆盖率,但是我刚刚和最新一次成功的记录对比了数值,发现目前数值有较大差距,我再排查一下,这个修改还存在问题。

@wangfenjin
Copy link
Owner

https://github.com/lukka/run-cmake

可以尝试下按照这个 readme 里面的更新 cmake

@wangfenjin
Copy link
Owner

类似 Codecov Report

这个 report 怎么也没了,看来是有些东西要更新了

@xujiajiadexiaokeai
Copy link
Author

我翻了一下ci记录,report确实失效一段时间了,我先尝试更新下cmake的部分。
然后刚刚提到的测试覆盖率数值的问题,我使用 ubuntu-22.04 跑下来覆盖率是与之前一致的。目前判断下来是在新的 ubuntu-24.04 中还存在一些问题,还需要一些时间排查,如果有新的进展我会更新上来。

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