dotnet/runtime

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

Open

#28.888 geöffnet am 6. März 2019

Auf GitHub ansehen
 (9 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)C# (5.445 Forks)batch import
area-Infrastructure-librariesenhancementhelp wanted

Repository-Metriken

Stars
 (17.886 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 12T 11h) (661 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche den aktuellen Ressourcen Einbindungsmechanismus für PartialFacades und NotSupported Assemblys im dotnet/runtime Buildsystem. Überprüfe die in der Issue vorgeschlagenen Änderungen: PartialFacades sollen standardmäßig keine Ressourcen enthalten und für NotSupported Assemblys soll eine eindeutige RESX Datei erforderlich sein. Sieh dir die MSBuild Targets und Props Dateien an, die die Ressourceneinbettung behandeln, wie z. B. die in den Verzeichnissen eng/ oder src/libraries/. Stelle sicher, dass Opt in Flags hinzugefügt werden und dass die Änderung bestehende Builds nicht beeinträchtigt. Berücksichtige Cross Kompilierungsszenarien.
Tech Stack
csharp
Domain
build system
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40