Skip to content

Commit c32ea38

Browse files
committed
Some log cleanup
1 parent c0d5824 commit c32ea38

1 file changed

Lines changed: 2 additions & 21 deletions

File tree

src/Tasks/CollectDeclaredReferencesTask.cs

Lines changed: 2 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,6 @@ public override bool Execute()
6161
AppDomain.CurrentDomain.AssemblyResolve += ResolveAssembly;
6262
try
6363
{
64-
Log.LogMessage(MessageImportance.Low, "OutputFile='{0}', TargetFrameworkMoniker='{1}', TargetPlatformMoniker='{2}', RuntimeIdentifier='{3}'",
65-
OutputFile, TargetFrameworkMoniker, TargetPlatformMoniker, RuntimeIdentifier);
66-
6764
List<DeclaredReference> declaredReferences = new();
6865

6966
if (References != null)
@@ -90,7 +87,7 @@ public override bool Execute()
9087
// Ignore references from packages. Those as handled later.
9188
if (reference.GetMetadata("NuGetPackageId").Length != 0)
9289
{
93-
Log.LogMessage(MessageImportance.Low, "Skipping Reference '{0}' because it comes from NuGet package '{1}' (handled via PackageReferences)", reference.ItemSpec, reference.GetMetadata("NuGetPackageId"));
90+
// Logs will be emitted for these references when processing the PackageReferences
9491
continue;
9592
}
9693

@@ -108,25 +105,15 @@ public override bool Execute()
108105
if (!string.IsNullOrEmpty(referenceHintPath) && File.Exists(referenceHintPath))
109106
{
110107
referencePath = Path.GetFullPath(referenceHintPath);
111-
Log.LogMessage(MessageImportance.Low, "Reference '{0}' resolved via HintPath to '{1}'", referenceSpec, referencePath);
112108
}
113109
else if (File.Exists(referenceSpec))
114110
{
115111
referencePath = Path.GetFullPath(referenceSpec);
116-
Log.LogMessage(MessageImportance.Low, "Reference '{0}' resolved via ItemSpec path to '{1}'", referenceSpec, referencePath);
117112
}
118113
else
119114
{
120115
var resolvedReference = ResolvedReferences.SingleOrDefault(rr => string.Equals(rr.GetMetadata("OriginalItemSpec"), referenceSpec, StringComparison.OrdinalIgnoreCase));
121116
referencePath = resolvedReference is null ? null : resolvedReference.ItemSpec;
122-
if (referencePath is not null)
123-
{
124-
Log.LogMessage(MessageImportance.Low, "Reference '{0}' resolved via ResolvedReferences to '{1}'", referenceSpec, referencePath);
125-
}
126-
else
127-
{
128-
Log.LogMessage(MessageImportance.Low, "Reference '{0}' could not be resolved to a file path", referenceSpec);
129-
}
130117
}
131118

132119
// If the reference is under the nuget package root, it's likely a Reference added in a package's props or targets.
@@ -141,7 +128,6 @@ public override bool Execute()
141128

142129
if (referencePath is not null)
143130
{
144-
Log.LogMessage(MessageImportance.Low, "Including Reference '{0}' with resolved path '{1}'", referenceSpec, referencePath);
145131
declaredReferences.Add(new DeclaredReference(referencePath, DeclaredReferenceKind.Reference, referenceSpec));
146132
}
147133
}
@@ -168,15 +154,12 @@ public override bool Execute()
168154
bool isTransitiveDependency = !string.IsNullOrEmpty(projectReference.GetMetadata("NuGetPackageId"));
169155
if (isTransitiveDependency)
170156
{
171-
// Ignore transitive project references since the project doesn't have direct control over them.
172-
Log.LogMessage(MessageImportance.Low, "Skipping ProjectReference '{0}' because it is a transitive dependency (NuGetPackageId='{1}')", projectReference.ItemSpec, projectReference.GetMetadata("NuGetPackageId"));
173157
continue;
174158
}
175159

176160
string projectReferenceAssemblyPath = Path.GetFullPath(projectReference.ItemSpec);
177161
string referenceProjectFile = projectReference.GetMetadata("OriginalProjectReferenceItemSpec");
178162

179-
Log.LogMessage(MessageImportance.Low, "Including ProjectReference '{0}' (project file: '{1}', assembly path: '{2}')", projectReference.ItemSpec, referenceProjectFile, projectReferenceAssemblyPath);
180163
declaredReferences.Add(new DeclaredReference(projectReferenceAssemblyPath, DeclaredReferenceKind.ProjectReference, referenceProjectFile));
181164
}
182165
}
@@ -187,7 +170,6 @@ public override bool Execute()
187170

188171
if (PackageReferences != null)
189172
{
190-
Log.LogMessage(MessageImportance.Low, "Processing {0} PackageReference(s)", PackageReferences.Length);
191173
Dictionary<string, PackageInfo> packageInfos = GetPackageInfos();
192174
foreach (ITaskItem packageReference in PackageReferences)
193175
{
@@ -201,7 +183,7 @@ public override bool Execute()
201183
if (!packageInfos.TryGetValue(packageReference.ItemSpec, out PackageInfo packageInfo))
202184
{
203185
// These are likely Analyzers, tools, etc.
204-
Log.LogMessage(MessageImportance.Low, "Skipping PackageReference '{0}' because it has no compile-time assemblies or build files (likely an Analyzer, tool, or content-only package)", packageReference.ItemSpec);
186+
Log.LogMessage(MessageImportance.Low, "Skipping PackageReference '{0}' because it has no compile-time assemblies (likely an Analyzer, tool, or content-only package)", packageReference.ItemSpec);
205187
continue;
206188
}
207189

@@ -215,7 +197,6 @@ public override bool Execute()
215197
continue;
216198
}
217199

218-
Log.LogMessage(MessageImportance.Low, "Including PackageReference '{0}' with {1} compile-time assembly(ies)", packageReference.ItemSpec, packageInfo.CompileTimeAssemblies.Count);
219200
foreach (string assemblyPath in packageInfo.CompileTimeAssemblies)
220201
{
221202
declaredReferences.Add(new DeclaredReference(assemblyPath, DeclaredReferenceKind.PackageReference, packageReference.ItemSpec));

0 commit comments

Comments
 (0)