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

feat(core): customizable Chromium executable path #9670

Merged
merged 4 commits into from
May 1, 2022

Conversation

Rongronggg9
Copy link
Contributor

@Rongronggg9 Rongronggg9 commented Apr 30, 2022

该 PR 相关 Issue / Involved issue

Close #

完整路由地址 / Example for the proposed route(s)

NOROUTE

新 RSS 检查列表 / New RSS Script Checklist

  • 新的路由 New Route
  • 文档说明 Documentation
    • 中文文档 CN
    • 英文文档 EN
  • 全文获取 fulltext
    • 使用缓存 Use Cache
  • 反爬/频率限制 anti-bot or rate limit?
    • 如果有, 是否有对应的措施? If yes, do your code reflect this sign?
  • 日期和时间 date and time
    • 可以解析 Parsed
    • 时区调整 Correct TimeZone
  • 添加了新的包 New package added
  • Puppeteer

说明 / Note

feat(core): customizable Chromium executable path

also build Chromium-bundled Docker image for arm/arm64

Signed-off-by: Rongrong <i@rong.moe>

arm64

arm/v7

also build Chromium-bundled Docker image for arm/arm64

Signed-off-by: Rongrong <i@rong.moe>
Signed-off-by: Rongrong <i@rong.moe>
@Rongronggg9 Rongronggg9 force-pushed the custom-chromium-path branch from 4037db0 to 8df5e48 Compare April 30, 2022 22:16
@Rongronggg9
Copy link
Contributor Author

Rongronggg9 commented Apr 30, 2022

It seems that the build caching issue still occurs from time to time. It should be a weird upstream issue so we can do nothing but only wait for upstream to fix it: moby/buildkit#2822 (comment)

But don't worry, this PR won't make it worse since the build flow of the ordinary image, whose cache goes into gha, did not change.

The build flow of the Chromium-bundled image can still re-use all caches from the ordinary build and from inline caches within the previous pushed image (inline cache seems not affected by the issue). Typical cache-all-hit build time is <40s, cache-miss build time is ~4.5min.

.github/workflows/test.yml Outdated Show resolved Hide resolved
docs/install/README.md Outdated Show resolved Hide resolved
docs/install/README.md Outdated Show resolved Hide resolved
docs/install/README.md Outdated Show resolved Hide resolved
Signed-off-by: Rongrong <i@rong.moe>
Signed-off-by: Rongrong <i@rong.moe>
@TonyRL TonyRL merged commit 7a6be9a into DIYgod:master May 1, 2022
@Rongronggg9 Rongronggg9 deleted the custom-chromium-path branch May 5, 2022 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants