dotnet/runtime

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

Aberta

#28.888 aberto em 6 de mar. de 2019

Ver no GitHubGuia do colaborador
 (9 comentários) (2 reações) (0 responsável)C# (5.445 forks)batch import
area-Infrastructure-librariesenhancementhelp wanted

Métricas do repositório

Stars
 (17.886 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Investigue o mecanismo atual de inclusão de recursos para assemblies PartialFacades e NotSupported no sistema de build do dotnet/runtime. Revise as alterações propostas na issue: fazer com que PartialFacades não incluam recursos por padrão e exigir um resx exclusivo para assemblies NotSupported. Examine os arquivos de destino e de propriedades do MSBuild que lidam com a incorporação de recursos, como aqueles nos diretórios eng/ ou src/libraries/. Certifique se de que os sinalizadores de opção (opt in) sejam adicionados e que a alteração não quebre as builds existentes. Considere cenários de compilação cruzada (cross compilation).
Pilha de tecnologia
csharp
Domain
build system
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-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40