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

ci: 避免每次从零安装 texlive,其他小改进 #683

Merged
merged 7 commits into from
Nov 27, 2023

Conversation

muzimuzhi
Copy link
Contributor

@muzimuzhi muzimuzhi commented Nov 19, 2023

最近 GitHub Actions 安装 texlive 经常失败,造成假的失败提醒。

这个 PR 把目前的手动安装 texlive 改为了使用 action teatimeguest/setup-texlive-action 安装 texlive。这个 action 会自动缓存 texlive 目录,减少从零安装 texlive 的次数。

其他的小改进可以从 commit message 里看到。

@Harry-Chen
Copy link
Member

README 里面有一个 badge URL 也要跟着 action 的名字改掉

@muzimuzhi
Copy link
Contributor Author

@Harry-Chen Badge URL 不用改,因为现在 github 用 workflow file name 作 URL 的最后一段(以前是用 workflow name)。我可以跟随 workflow 更名,改一下 link text。

注意:Win 下需要安装 `wintools.windows` 以使用 texlive 提供的 unzip,
代替 Git for Windows 提供的。后者的行为与标准 zlib 提供的不一致。
这避免了在 win 下测试时输出
    build.lua:23: module 'socket.http' not found:

出于安全考虑,LuaTeX v1.17.0 起默认禁止加载 `socket` 库。
开启需使用 `luatex --socket` 或 `luatex --shell-escape`。
见 https://tug.org/~mseven/luatex.html#luasocket
@muzimuzhi
Copy link
Contributor Author

虽然只在 ./ctex 下有测试文件,但是因为用多个引擎跑了测试集,xeCJK 等包确实测试到了一些。所以我重新修改了 workflow name。

把它改短,是为了在 check list 中留出宽度显示 job name 和 trigger event(如 pushpull_request)。

  • 改之前
    image
  • 改之后(现在)
    image

@muzimuzhi muzimuzhi merged commit e0a2a8f into CTeX-org:master Nov 27, 2023
3 checks passed
@muzimuzhi muzimuzhi deleted the ci/cache-texlive branch November 27, 2023 00:33
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