dotnet/runtime

Properly factor DllImports into Common\src\Interop

Offen

#28.035 geöffnet am 30.11.2018

Auf GitHub ansehenContributor Guide
 (23 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (5.445 Forks)batch import
area-Metaenhancementgood first issuehelp wantedtracking

Repository-Metriken

Stars
 (17.886 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

We now have a bunch of libraries with their own DllImports, whereas the expected approach is that all DllImports in product src should be included from src\Common\src\Interop, appropriately factored by platform and library, and then those source files included into the relevant assemblies that need them.

  • Microsoft.Diagnostics.Tracing.EventSource.Redist
  • System.Data.SqlClient
  • System.Diagnostics.EventLog
  • System.DirectoryServices
  • System.DirectoryServices.AccountManagement
  • System.DirectoryServices.Protocols
  • System.Drawing.Common
  • System.IO.Compression
  • System.IO.Compression.Brotli
  • System.IO.Ports
  • System.Management
  • System.Reflection.Metadata
  • System.Runtime.Caching
  • System.Security.Cryptography.Csp
  • System.Security.Cryptography.Pkcs
  • System.Security.Cryptography.X509Certificates

This all needs to be refactored and dedup'd.

https://github.com/dotnet/corefx/blob/master/Documentation/coding-guidelines/interop-guidelines.md

Contributor Guide

Research-Richtung
Überprüfen Sie die im Issue verlinkten Interop Richtlinien. Identifizieren Sie dann für jede nicht abgehakte Bibliothek alle DllImport Deklarationen im Quellcode, teilen Sie sie nach Plattform und Bibliothek in geeignete Dateien unter src/Common/src/Interop auf und binden Sie diese Quelldateien in die entsprechenden Assemblys ein.
Tech Stack
csharp
Domain
developer experience
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
GitC#.NET interop
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45