mozilla/sccache

Add a higher-level interface to RunCommand

Open

#95 aperta il 28 mar 2017

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Rust (677 fork)github user discovery
help wanted

Metriche repository

Star
 (7329 star)
Metriche merge PR
 (Merge medio 28g 22h) (11 PR mergiate in 30 g)

Descrizione

@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.

Guida contributor

Direzione di ricerca
Esamina la struttura RunCommand esistente e il suo utilizzo nelle invocazioni del compilatore per identificare i pattern ripetitivi di env clear e envs. Proponi un'astrazione di livello superiore che incapsuli la configurazione dell'ambiente.
Tech stack
rust
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50