demo-up.sh builds the lobby without LUCKPERMS_JAR_URL, which its Dockerfile refuses #21

New Issue

2026-07-31T05:13:26+09:00

FLYEMOJ1 commented

2026-07-31 05:13:26 +09:00

(Migrated from github.com)

deploy/demo-up.sh 的现场构建路径（deploy/images/ 无预打包 tar 时的 docker build 分支）给 lobby 传的 build-arg 只有 Paper 的两个；而 80a29ba 起 deploy/lobby/Dockerfile 的 RUN 守卫把 LUCKPERMS_JAR_URL 设为必填、缺了直接 exit 1（ERROR: --build-arg LUCKPERMS_JAR_URL=<luckperms bukkit jar> is required）。所以这条路径今天在 lobby 一步必然失败。

80a29ba 让 LuckPerms 成为必填时更新了 bootstrap.sh 的调用方（build_game_stack 一直在传），demo-up.sh 漏了——它上一次被改还是 b84debf。
预打包 tar 路径不受影响。
修法方向：demo-up 像 bootstrap.sh 的 resolve_game_jars 一样解析 luckperms_latest_jar（metadata.luckperms.net）并传入，允许 LUCKPERMS_JAR_URL env 覆盖——和它现有的 PAPER_JAR_URL override 模式一致。

发现于 #20 施工中（给两条 docker build 补 PAPER_JAR_SHA256 时）。

`deploy/demo-up.sh` 的现场构建路径（`deploy/images/` 无预打包 tar 时的 `docker build` 分支）给 lobby 传的 build-arg 只有 Paper 的两个；而 80a29ba 起 `deploy/lobby/Dockerfile` 的 RUN 守卫把 `LUCKPERMS_JAR_URL` 设为必填、缺了直接 `exit 1`（`ERROR: --build-arg LUCKPERMS_JAR_URL=<luckperms bukkit jar> is required`）。所以这条路径今天在 lobby 一步必然失败。 - 80a29ba 让 LuckPerms 成为必填时更新了 `bootstrap.sh` 的调用方（`build_game_stack` 一直在传），demo-up.sh 漏了——它上一次被改还是 b84debf。 - 预打包 tar 路径不受影响。 - 修法方向：demo-up 像 bootstrap.sh 的 `resolve_game_jars` 一样解析 `luckperms_latest_jar`（metadata.luckperms.net）并传入，允许 `LUCKPERMS_JAR_URL` env 覆盖——和它现有的 `PAPER_JAR_URL` override 模式一致。发现于 #20 施工中（给两条 docker build 补 `PAPER_JAR_SHA256` 时）。

Lemon-miaow commented

2026-09-24 12:02:23 +09:00

(Migrated from github.com)

按修法方向收口（f5a76cf，批 40 单起点化）：demo-up.sh 不再带自己的 docker build 逻辑，改为 bootstrap 的薄包装——base + game stack 全部由安装器单一来源构建，LUCKPERMS_JAR_URL 由 installer 的 resolve_game_jars 统一解析注入。复核：grep -n 'docker build' deploy/demo-up.sh 0 命中；真机重跑 demo-up 全绿（批 40 T1–T3）。

按修法方向收口（`f5a76cf`，批 40 单起点化）：`demo-up.sh` 不再带自己的 `docker build` 逻辑，改为 bootstrap 的薄包装——base + game stack 全部由安装器单一来源构建，`LUCKPERMS_JAR_URL` 由 installer 的 `resolve_game_jars` 统一解析注入。复核：`grep -n 'docker build' deploy/demo-up.sh` 0 命中；真机重跑 demo-up 全绿（批 40 T1–T3）。

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: FelisMC/Felis#21