dotnet/runtime

Facades and NotSupported assemblies are bloated with resources they don't use

Open

#28 888 ouverte le 6 mars 2019

Voir sur GitHub
 (9 commentaires) (2 réactions) (0 assignés)C# (5 445 forks)batch import
area-Infrastructure-librariesenhancementhelp wanted

Métriques du dépôt

Stars
 (17 886 stars)
Métriques de merge PR
 (Merge moyen 12j 11h) (661 PRs mergées en 30 j)

Description

Often times we have builds that cross-compile and have very different implementations. Our resource conventions don't force folks to think about this, so the resources are often a superset of all configurations.

We should change the defaults so that this isn't the case.

I propose the following:

  1. By default PartialFacades should not include any resources, unless a project opts-in. Folks should discover this relatively easily as SR.Foo will fail to work in the partial source. They can fix it by setting a flag and potentially providing specific resources for that build of the library.
  2. By default NotSupported assemblies require a unique resx. We can change the path convention to something like Resources\NotSupported.resx instead of strings.resx

Across all our builds this will save over a MB of unused resources.

Guide contributeur

Direction de recherche
Étudiez le mécanisme actuel d'inclusion des ressources pour les assemblys PartialFacades et NotSupported dans le système de build de dotnet/runtime. Examinez les modifications proposées dans l'issue : faire en sorte que les PartialFacades n'incluent pas de ressources par défaut et exiger un fichier resx unique pour les assemblys NotSupported. Consultez les fichiers cibles et de propriétés MSBuild qui gèrent l'intégration des ressources, comme ceux dans les répertoires eng/ ou src/libraries/. Assurez vous que des indicateurs d'opt in sont ajoutés et que le changement ne casse pas les builds existantes. Prenez en compte les scénarios de cross compilation.
Stack technique
csharp
Domaine
build system
Type d'issue
Refactorisation
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40