cake-build/cake

Tool.Run should make stdout and stderr available on failure

Ouverte

#2 460 ouverte le 24 janv. 2019

Voir sur GitHubGuide contributeur
 (4 commentaires) (2 réactions) (0 personne assignée)C# (738 forks)batch import
Help wantedImprovementUp-for-grabs

Métriques du dépôt

Stars
 (3 742 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
La demande est de modifier la méthode Tool.Run dans src/Cake.Core/Tooling/Tool.cs pour capturer stdout et stderr avant de lever une exception. Actuellement, à la ligne 119, ProcessExitCode est appelé, et à la ligne 139, une simple CakeException est levée. L'amélioration consiste à créer une nouvelle classe d'exception CakeProcessException qui inclut le code de sortie, stdout et stderr. L'implémentation doit capturer la sortie/erreur standard du processus. Une approche suggérée est d'utiliser ProcessStartInfo.RedirectStandardOutput et RedirectStandardError avant de démarrer le processus, puis de lire les flux de sortie avant de vérifier le code de sortie. La nouvelle exception doit être levée à la place de CakeException. Il s'agit d'un changement bien défini avec des références de fichiers concrètes.
Stack technique
csharp
Domaine
build system
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70