dotnet/roslyn

[MSBuildWorkspace] ProjectReferences vs. AllProjectReferences return the same count despite projects not existing in the solution.

Offen

#46.113 geöffnet am 17.07.2020

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C# (4.257 Forks)batch import
Area-IDEConcept-Continuous ImprovementIDE-MSBuildWorkspacehelp wanted

Repository-Metriken

Stars
 (20.414 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (256 gemergte PRs in 30 T)

Beschreibung

Version Used: master, 16.6.0, etc

Steps to Reproduce:

  1. Create a project A and a project B
  2. Have A reference B
  3. Create a solution X and add only A to it
  4. Use MSBuildWorkspace and OpenSolutionAsync()
  5. Compare A's ProjectReferences to AllProjectReferences

Expected Behavior:

They should be different, because solution X only has A in it.

Actual Behavior:

They are the same because the way a solution is loaded causes project references to be loaded into it even if they're not actually in the solution. See here:

https://github.com/dotnet/roslyn/blob/2a69ac38995a07f4333158b76d2f6e780868401a/src/Workspaces/Core/MSBuild/MSBuild/MSBuildProjectLoader.Worker.cs#L138-L172

Contributor Guide

Research-Richtung
Überprüfen Sie den verknüpften Code in MSBuildProjectLoader.Worker.cs, Zeilen 138-172, um zu verstehen, wie Projektverweise geladen werden. Vergleichen Sie dann die Eigenschaften ProjectReferences und AllProjectReferences, um zu sehen, warum sie dieselbe Anzahl zurückgeben, wenn ein Projektverweis nicht in der Projektmappe enthalten ist.
Tech Stack
csharp
Domain
build system
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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
GitMSBuild
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70