cake-build/cake

Tool.Run should make stdout and stderr available on failure

Open

#2,460 建立於 2019年1月24日

在 GitHub 查看
 (4 留言) (2 反應) (0 負責人)C# (738 fork)batch import
Help wantedImprovementUp-for-grabs

倉庫指標

Star
 (3,742 star)
PR 合併指標
 (平均合併 1天 1小時) (30 天內合併 25 個 PR)

描述

Tool.Run and related should make stdout and stderr available so that some analysis is possible on failure. Currently it calls ProcessExitCode which throws a simple CakeException.

Ideally, it would capture stdout and stderr and throw a CakeProcessException, or similar, that has additional properties for exit code, stdout and stderr.

貢獻者指南

研究方向
請求修改 src/Cake.Core/Tooling/Tool.cs 中的 Tool.Run 方法，在拋出異常前捕獲 stdout 和 stderr。目前，在第 119 行呼叫 ProcessExitCode，並在第 139 行拋出一個簡單的 CakeException。改進方案是建立一個新的異常類別 CakeProcessException，其中包含退出碼、stdout 和 stderr。實作應在檢查退出碼之前，使用 ProcessStartInfo.RedirectStandardOutput 和 RedirectStandardError 重新導向標準輸出/錯誤流，然後讀取輸出流。應拋出新的異常而非 CakeException。這是一個有明確檔案參考的具體變更。
技術棧
csharp
領域
build system
議題類型
功能
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-2 天
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
活躍
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
清晰
前置要求
C#
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
70