cake-build/cake

Tool.Run should make stdout and stderr available on failure

Aperta

#2460 aperta il 24 gen 2019

Vedi su GitHubGuida contributor
 (4 commenti) (2 reazioni) (0 assegnatari)C# (738 fork)batch import
Help wantedImprovementUp-for-grabs

Metriche repository

Star
 (3742 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
La richiesta è di modificare il metodo Tool.Run in src/Cake.Core/Tooling/Tool.cs per catturare stdout e stderr prima di lanciare un'eccezione. Attualmente, alla riga 119 viene chiamato ProcessExitCode e alla riga 139 viene lanciata una semplice CakeException. Il miglioramento consiste nel creare una nuova classe di eccezione CakeProcessException che includa il codice di uscita, stdout e stderr. L'implementazione dovrebbe catturare l'output/errore standard dal processo. Un approccio suggerito è utilizzare ProcessStartInfo.RedirectStandardOutput e RedirectStandardError prima di avviare il processo, quindi leggere i flussi di output prima di controllare il codice di uscita. La nuova eccezione dovrebbe essere lanciata al posto di CakeException. Si tratta di una modifica ben definita con riferimenti concreti ai file.
Tech stack
csharp
Dominio
build system
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70