dotnet/msbuild

IMSBuildGlob should have a method that expands globs

Open

#1.908 aberto em 24 de mar. de 2017

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C# (1.364 forks)batch import
Feature - Globbinghelp wantedtriaged

Métricas do repositório

Stars
 (5.062 stars)
Métricas de merge de PR
 (Mesclagem média 11d 7h) (125 fundiu PRs em 30d)

Description

IMSBuildGlob currently knows how to match strings against itself. It would be nice if it also had an ExpandToFiles method which hit the disk and expanded the glob into matching files, with the same behaviour that the engine is internally using. We didn't expose this functionality because there's no outside asks for it yet.

Implementation details:

  • Since expansion is slow, the results should be cached, and maybe the method should also have an async counterpart
  • A very easy implementation is to just call EngineFileUtilities.GetFileList. However, that method needlessly parses the glob again. A better implementation is to rework the internal static methods to also accept an already parsed glob. This poses some complications due to the different validation checks done by EngineFileUtilities.GetFileList and FileMatcher.GetFiles
  • Behaviour on illegal filespecs (as per the engine today):
    • EngineFileUtilities.GetFileList returns back the original illegal filespec
    • FileMatcher.GetFiles return back the original illegal filespec if it's not excluded, otherwise returns empty

Guia do colaborador

Direção de pesquisa
Estude a interface IMSBuildGlob e a classe FileMatcher. Entenda como EngineFileUtilities.GetFileList funciona e como implementar um método ExpandToFiles com cache.
Pilha de tecnologia
csharp
Domain
build system
Tipo Issue
Funcionalidade
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-2 dias
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 internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
45