Skip to content

fix(builder): replace dead linglong.space doc URLs in error messages - #2182

Open
muzimu217 wants to merge 1 commit into
OpenAtom-Linyaps:masterfrom
muzimu217:fix/builder-doc-url-errors
Open

muzimu217 wants to merge 1 commit into
OpenAtom-Linyaps:masterfrom
muzimu217:fix/builder-doc-url-errors

Conversation

@muzimu217

Copy link
Copy Markdown
Contributor

Split out of #2145 so the docs and code changes can be reviewed independently. Supersedes #1974, the original PR for this fix.

问题

ll-builder 有两处错误提示指向旧文档站 https://linglong.space/guide/ll-builder/modules.html,该域名已无法解析,用户看到提示去查文档会直接失败:

  • apps/ll-builder/src/main.cpp:配置 binary modules 时的报错
  • libs/linglong/src/linglong/builder/linglong_builder.cpp:$appid.install 废弃提示

修改

两处 URL 统一替换为当前文档站对应页面 https://linyaps.org.cn/guide/building/modules.html。共 2 行,均为字符串字面量,不涉及任何逻辑。

验证

  • 旧域名已死、新页面活着:
$ curl -o /dev/null -s -w '%{http_code}\n' https://linyaps.org.cn/guide/building/modules.html
200

(linglong.space 已无解析,连接直接失败)

  • 全仓搜索 linglong.space:改动前仅这 2 处,改动后为 0
  • clang-format(22.1.8,与仓库 pre-commit 配置同版本)对两个文件零 diff
  • 未在本地做完整编译:改动是纯字符串字面量替换,完整构建交给上游 CI 矩阵

我是外部贡献者,该 PR 可能需要组织成员批准 CI(needs-ok-to-test)。恳请有空的维护者帮忙跑一下 CI,谢谢!

linglong.space no longer resolves (NXDOMAIN). Point the modules guide
URL in ll-builder error messages to the current docs site
(linyaps.org.cn).

Supersedes OpenAtom-Linyaps#1974.

Signed-off-by: muzimu217 <1278844978@qq.com>
@deepin-ci-robot

Copy link
Copy Markdown
Collaborator

Hi @muzimu217. Thanks for your PR.

I'm waiting for a OpenAtom-Linyaps member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@codecov

codecov Bot commented Oct 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
apps/ll-builder/src/main.cpp 0.00% <ø> (ø)
...linglong/src/linglong/builder/linglong_builder.cpp 10.81% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@muzimu217

Copy link
Copy Markdown
Contributor Author

Hi @Oyami-Srk — same request for this new one: could a trusted user run /ok-to-test? This is the code half split out of #2145 (two dead doc-URLs in builder error messages, 2 files, +2/−2). Build matrix was green when CI ran on the earlier head. Thanks!

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