dotnet/msbuild

IMSBuildGlob should have a method that expands globs

Open

#1 908 ouverte le 24 mars 2017

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C# (1 364 forks)batch import
Feature - Globbinghelp wantedtriaged

Métriques du dépôt

Stars
 (5 062 stars)
Métriques de merge PR
 (Merge moyen 11j 7h) (125 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Étudiez l'interface IMSBuildGlob et la classe FileMatcher. Comprenez le fonctionnement de EngineFileUtilities.GetFileList et comment implémenter une méthode ExpandToFiles avec mise en cache.
Stack technique
csharp
Domaine
build system
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#MSBuild internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45