Warn about System.Web, don't reject (for now)

dotnet · May 4, 2020 · 00c8a64 · 00c8a64
1 parent 0ab29a7
commit 00c8a64
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/MSBuild.Abstractions/MSBuildWorkspace.cs b/src/MSBuild.Abstractions/MSBuildWorkspace.cs
@@ -236,6 +236,11 @@ private bool IsSupportedProjectType(MSBuildProjectRootElement root)
                 return false;
             }
 
+            if (root.ItemGroups.Any(ig => ig.Items.Any(ProjectItemHelpers.IsReferencingSystemWeb)))
+            {
+                Console.WriteLine($"{root.FullPath} contains a reference to System.Web, which is not supported on .NET Core. You may have significant work ahead of you to fully port this project.");
+            }
+
             // Lots of wild old project types have project type guids that the old project system uses to light things up!
             // Also some references that are incompatible.
             var projectType = GetProjectSupportType(root);
@@ -289,8 +294,7 @@ static void PrintGuidMessage(IEnumerable<string> allSupportedProjectTypeGuids, I
 
             static ProjectSupportType GetProjectSupportType(MSBuildProjectRootElement root)
             {
-                if (root.PropertyGroups.Any(pg => pg.Properties.Any(ProjectPropertyHelpers.IsLegacyWebProjectTypeGuidsProperty))
-                    || root.ItemGroups.Any(ig => ig.Items.Any(ProjectItemHelpers.IsReferencingSystemWeb)))
+                if (root.PropertyGroups.Any(pg => pg.Properties.Any(ProjectPropertyHelpers.IsLegacyWebProjectTypeGuidsProperty)))
                 {
                     return ProjectSupportType.LegacyWeb;
                 }