mozilla/sccache

Add a higher-level interface to RunCommand

Open

#95 ouverte le 28 mars 2017

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Rust (677 forks)github user discovery
help wanted

Métriques du dépôt

Stars
 (7 329 stars)
Métriques de merge PR
 (Merge moyen 28j 22h) (11 PRs mergées en 30 j)

Description

@alexcrichton pointed out that the compiler implementations had some repetitive and potentially error-prone code for spawning processes with my changes in #92. Specifically for each compiler invocation it has to call cmd.env_clear().envs(...) to properly set the environment. We should add a higher-level interface here so that it's harder to make a mistake in this code.

Guide contributeur

Direction de recherche
Examinez la structure RunCommand existante et son utilisation dans les invocations du compilateur pour identifier les motifs répétitifs de env clear et envs. Proposez une abstraction de plus haut niveau qui encapsule la configuration de l'environnement.
Stack technique
rust
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50