Windows builds hang when the HEAD commit message is large (Invoke-Git deadlock)
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- PowerShell
- Star
- 145
- Fork
- 27
- Merge trung bình
- 10 giờ 16 phút
- Pull request đã merge (30 ngày)
- 34
Mô tả
Part of #120. Found while verifying the #162 merge.
Symptom
On Windows, Set-BuildEnvironment hangs forever when the HEAD commit message is large. Nothing
is printed, no error, no timeout — the build never starts.
This is currently happening on main. The push-triggered CI run for 9720cb7 hung both Windows
legs for 27+ minutes with zero output (ubuntu and macOS passed); I cancelled it. Every local
./build.ps1 on Windows hangs the same way, because build.ps1 calls Set-BuildEnvironment -Force before Invoke-psake.
Root cause
BuildHelpers\Invoke-Git (2.0.16) redirects both output streams and then waits for the process
to exit before reading them:
$pinfo.RedirectStandardError = $RedirectStandardError # $true
$pinfo.RedirectStandardOutput = $RedirectStandardOutput # $true
...
$null = $p.Start()
$p.WaitForExit() # <-- blocks here
...
$stdout = $p.StandardOutput.ReadToEnd() # <-- never reached
When git writes more than the pipe buffer holds, git blocks waiting for someone to drain the
pipe, and Invoke-Git blocks waiting for git to exit. Neither can proceed. It is the textbook
WaitForExit-before-ReadToEnd deadlock described in the
ProcessStartInfo.RedirectStandardOutput remarks.
Get-BuildVariable calls it as Invoke-Git log --format=%B -n 1 <sha> to populate
$env:BHCommitMessage, so the payload is the full commit message body.
Reproduction
Measured on Windows 11, PowerShell 7.6.4, BuildHelpers 2.0.16, against real commits on main:
| Commit | Message size | Result |
|---|---|---|
f726f48 |
2617 bytes | returns |
64b7207 |
5498 bytes | returns |
9720cb7 |
5757 bytes | hangs |
Import-Module BuildHelpers
Invoke-Git -Arguments 'log --format=%B -n 1 9720cb7' # never returns on Windows
Linux and macOS have a larger pipe buffer and do not hit the threshold at these sizes, which is
why only the Windows legs hang.
Why CI did not catch it before merge
On a pull_request event, GitHub checks out a synthetic merge commit whose message is short
(Merge <sha> into <sha>). The real squash message only becomes HEAD on the push to main.
Every PR will be green and then hang main if its squashed body is large enough.
This repository writes long PR bodies and squash-merges them verbatim, so the trigger is not
exotic — it is the house style. #163 and #165 both have bodies over the threshold today.
Consumer impact
This is not only a repo-internal problem. Initialize-PSBuild calls Set-BuildEnvironment, so
any consumer building on Windows hangs when their own HEAD commit message crosses the threshold.
A hang with no output is a bad failure mode to ship at 1.0.0.
Options
- Stop passing the payload through
Invoke-Git. Populate$env:BHCommitMessageourselves
before callingSet-BuildEnvironment, or truncate it. Narrow, immediate, ours to control - Fix upstream. Read the streams asynchronously (
BeginOutputReadLine, orReadToEnd
beforeWaitForExit) and PR it to BuildHelpers, then raise theRequiredModulesfloor once
released. Correct fix, slowest path - Trim squash commit messages at merge time. Pure process change, no code — but it depends
on every future merge remembering, and it does nothing for consumers - Drop the BuildHelpers dependency for this variable. Largest change; worth naming only
because #120 already asks what 1.0.0 should depend on
Options 1 and 2 are not exclusive: 1 unblocks now, 2 fixes it for everyone.
Immediate mitigation
Until this is fixed, squash-merge with a short commit body on this repo. The full PR
description stays on the pull request either way.
Done when
./build.ps1 and Windows CI complete regardless of HEAD commit message size, with a regression
test covering a message larger than the pipe buffer.
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với build.ps1 và đường dẫn Set-BuildEnvironment, sau đó kiểm tra BuildHelpers\Invoke-Git và Get-BuildVariable để xác nhận cách đọc thông điệp commit. Tái hiện hiện tượng treo bằng Invoke-Git log --format=%B -n 1 9720cb7 trên Windows và xem lại #120 trước khi chọn một trong các tùy chọn giảm thiểu được liệt kê. Được xem là hoàn tất khi ./build.ps1 và Windows CI chạy hoàn tất với các thông điệp HEAD lớn, cùng một bài kiểm thử hồi quy bao quát thông điệp lớn hơn bộ đệm pipe.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- git, powershell
- Lĩnh vực
- build-system, ci-cd, tooling
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Sôi nổi
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 50/100