Skip to content

stage-index: drop the wrapper nothing calls - #238

Merged
Zakkaus merged 1 commit into
masterfrom
chore/drop-dead-code
Aug 10, 2026
Merged

stage-index: drop the wrapper nothing calls#238
Zakkaus merged 1 commit into
masterfrom
chore/drop-dead-code

Conversation

@Zakkaus

@Zakkaus Zakkaus commented Aug 10, 2026

Copy link
Copy Markdown
Member

仓库盘点找到的低风险清理,两项。

build/stage-index.pyportage_restrict()portage_policy(overlay)[0]
一层包装,全仓库搜索只有定义与一条测试呼叫,生产路径没有呼叫点。移除包装,那条
测试改为直接呼叫 portage_policy()

docs/dependency-closure.md 里三处 #L<行号> 锚点已经全部偏移:runtime_atoms
写 111 实际 122、runtime_closure 写 172 实际 177、check 写 201 实际 191。
行号锚点在下一次改动就失效,改成只指档案。

测试

守的行为没变:把 MetadataUnavailable 那条 fail-closed 拿掉之后
tests/test-stage-index.py 仍然转红(rc=1)。本机全套 Python、Shell、JavaScript
测试、check-copypy_compilecheck-commits 通过。

`portage_restrict()` 只是 `portage_policy(overlay)[0]` 的一层包装,生产路径
没有任何呼叫点,只有一条测试透过它验证仓库路径解析不到时会 fail-closed。

移除包装,那条测试改为直接呼叫 `portage_policy()`,守的行为不变。

顺带清掉 `docs/dependency-closure.md` 里三处指向源码的行号锚点。行号在下一
次改动就失效,实测三处已经全部偏移,改成只指档案。
@Zakkaus
Zakkaus merged commit abb17c7 into master Aug 10, 2026
7 checks passed
@Zakkaus
Zakkaus deleted the chore/drop-dead-code branch August 10, 2026 16:02
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.

1 participant