mozilla/sccache

Add a higher-level interface to RunCommand

Open

#95 aberto em 28 de mar. de 2017

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Rust (677 forks)github user discovery
help wanted

Métricas do repositório

Stars
 (7.329 stars)
Métricas de merge de PR
 (Mesclagem média 28d 22h) (11 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Examine a estrutura RunCommand existente e seu uso em invocações do compilador para identificar os padrões repetitivos de env clear e envs. Proponha uma abstração de nível superior que encapsule a configuração do ambiente.
Pilha de tecnologia
rust
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Rust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50