ci: mcpp 0.0.101 → 0.0.102 (windows scan-deps cmdline ceiling, mcpp#261)#113
Closed
Sunrisepeak wants to merge 1 commit into
Closed
ci: mcpp 0.0.101 → 0.0.102 (windows scan-deps cmdline ceiling, mcpp#261)#113Sunrisepeak wants to merge 1 commit into
Sunrisepeak wants to merge 1 commit into
Conversation
…261)
The windows workspace leg builds a package's own TUs from under the
registry's xpkgs path (~124 chars) rather than a ~23-char checkout, so
the clang scan-deps command — wrapped in `cmd /c` only to redirect its
P1689 JSON — crossed cmd.exe's 8191-char ceiling on the vendored-opencv
consumer (opencv-module{,-dnn}). mcpp 0.0.102 drops the wrapper via
clang-scan-deps -o and extends #247's response-file fallback to every
$local_includes rule, restoring the CreateProcess 32767 ceiling.
Bump min_mcpp/latest_mcpp together with the CI pin per the index.toml
version-contract rule. Also carries #257 (Clang purview-include depfile),
#258 (OS-conditional [build].flags) and #254 (per-OS splice on target).
Member
Author
|
Folded into #111: the 0.0.102 floor bump exists because opencv 0.0.8's windows leg needs mcpp#261, and #111 already deletes the old compat.opencv members that a separate full regression here would waste windows-build time on. #111 now carries the validate.yml + index.toml bump together with the descriptor, so one run builds the new single-repo members on the right mcpp. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
索引侧把 CI 的 mcpp pin 与版本契约从 0.0.101 抬到 0.0.102。
背景
windows workspace 腿在把包当依赖从注册表消费时,包自己的 TU 落在 xpkgs 路径下(
…\.mcpp\.xlings\data\xpkgs\mcpplibs-x-opencv\0.0.7\opencv-m-0.0.7\,约 124 字符),而不是包自己 CI 里的D:\a\opencv-m\opencv-m\(约 23 字符)。每个-I都多背 ~100 字符前缀,vendored-opencv 消费者(opencv-module/opencv-module-dnn)的 clang scan-deps 命令因此越过 cmd.exe 的 8191 字符上限,报The command line is too long.。那个
cmd /c只是为了用 shell 重定向> $out拿 P1689 JSON。mcpp 0.0.102(mcpp#261) 改用clang-scan-deps -o去掉 wrapper,并把 #247 的 response-file 兜底扩展到每条带 $local_includes 的规则,上限从 8191 回到 CreateProcess 的 32767。opencv-m 自己 CI 的 windows 腿是绿的,正是因为它的路径前缀短——这不是包代码问题,而是路径深度触发的引擎侧上限。
改动
.github/workflows/validate.yml:MCPP_VERSION与三个矩阵mcpp_version0.0.101 → 0.0.102,并补版本说明注释。index.toml:min_mcpp/latest_mcpp同步抬到 0.0.102(按契约与 CI pin 同步)。0.0.102 同时带 #257(Clang purview-include depfile 追踪)、#258(OS 条件
[build].flags)、#254(per-OS splice 按 target 而非 host)。编辑了
validate.yml,因此本 PR 会触发全量 workspace 构建 —— 正好作为 0.0.102 的一次全矩阵回归,验证 windows 腿恢复绿色。合入后 rebase #111(opencv 0.0.7 单仓库描述符)。