dotnet/runtime

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

Open

#28.888 aperta il 6 mar 2019

Vedi su GitHub
 (9 commenti) (2 reazioni) (0 assegnatari)C# (5445 fork)batch import
area-Infrastructure-librariesenhancementhelp wanted

Metriche repository

Star
 (17.886 star)
Metriche merge PR
 (Merge medio 12g 11h) (661 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Indaga sul meccanismo corrente di inclusione delle risorse per gli assembly PartialFacades e NotSupported nel sistema di build di dotnet/runtime. Rivedi le modifiche proposte nell'issue: fare in modo che i PartialFacades non includano risorse per impostazione predefinita e richiedere un file resx univoco per gli assembly NotSupported. Esamina i file di destinazione e di proprietà MSBuild che gestiscono l'incorporamento delle risorse, come quelli nelle directory eng/ o src/libraries/. Assicurati che vengano aggiunti flag di opt in e che la modifica non interrompa le build esistenti. Considera gli scenari di cross compilazione.
Tech stack
csharp
Dominio
build system
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40