dotnet/roslyn

Consider having CopyRefAssembly inherit from Copy

Open

#19.103 aberto em 28 de abr. de 2017

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)C# (4.257 forks)batch import
Area-CompilersBughelp wanted

Métricas do repositório

Stars
 (20.414 stars)
Métricas de merge de PR
 (Mesclagem média 5d 21h) (286 fundiu PRs em 30d)

Description

The new CopyRefAssembly task has logic to determine whether a reference assembly is logically identical by comparing MVIDs of the source and destination files. If the MVIDs do not match, it copies the source assembly to the destination path.

As currently implemented, this copy doesn't have all of the bells and whistles of the Copy task, which supports retries and special-cases a number of scenarios.

CopyRefAssembly could inherit from Copy, override Execute(), do its up-to-date checks, and then after deciding to copy, forward to base.Execute() to get all of the additional magic.

This came up when discussing https://github.com/Microsoft/msbuild/issues/1986 with @jeffkl.

@jcouv -- something to think about. Doesn't block the initial implementation, since no MSBuild change would be required to pick this up.

Guia do colaborador

Direção de pesquisa
Investigue a implementação da tarefa CopyRefAssembly e a tarefa Copy no código fonte do MSBuild. O objetivo é refatorar CopyRefAssembly para herdar de Copy e encaminhar a execução para base.Execute() após verificações personalizadas de atualização. Revise a discussão vinculada e o problema relacionado (Microsoft/msbuild#1986) para contexto. Examine o método Execute existente e a lógica de cópia para garantir compatibilidade.
Pilha de tecnologia
csharp
Domain
build systemdeveloper experience
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.
3
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.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#MSBuild task implementation
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30