Revert "Enabling ConvertPdbsToWindowsPdbs (#8007)"

This reverts commit a661399.
dotnet · Oct 10, 2021 · 192d2d0 · 192d2d0
1 parent 020ac68
commit 192d2d0
Show file tree

Hide file tree

Showing 6 changed files with 5 additions and 22 deletions.
diff --git a/eng/publishing/v3/publish-assets.yml b/eng/publishing/v3/publish-assets.yml
@@ -103,7 +103,6 @@ jobs:
           /p:DotNetBuildsPublicChecksumsUriBase64='$(dotnetbuilds-public-container-checksum-uri-base64)'
           /p:DotNetBuildsInternalUriBase64='$(dotnetbuilds-internal-container-uri-base64)'
           /p:DotNetBuildsInternalChecksumsUriBase64='$(dotnetbuilds-internal-container-checksum-uri-base64)'
-          /p:ConvertPortablePdbsToWindowsPdbs='true'
     - template: /eng/common/templates/steps/publish-logs.yml
       parameters:
         StageLabel: '${{ parameters.stageName }}'

diff --git a/src/Microsoft.DotNet.Arcade.Sdk/tools/SdkTasks/PublishArtifactsInManifest.proj b/src/Microsoft.DotNet.Arcade.Sdk/tools/SdkTasks/PublishArtifactsInManifest.proj
@@ -108,7 +108,6 @@
       <InternalChecksumsFeedKey>$(InternalChecksumsAzureAccountKey)</InternalChecksumsFeedKey>
       <AllowFeedOverrides Condition="'$(AllowFeedOverrides)' == ''">false</AllowFeedOverrides>
       <UseStreamingPublishing Condition="'$(UseStreamingPublishing)' == ''">false</UseStreamingPublishing>
-      <ConvertPortablePdbsToWindowsPdbs Condition="'$(ConvertPortablePdbsToWindowsPdbs)' == ''">false</ConvertPortablePdbsToWindowsPdbs>
       <ArtifactsBasePath Condition="'$(ArtifactsBasePath)' == ''">$(BlobBasePath)</ArtifactsBasePath>
       <NonStreamingPublishingMaxClients Condition="'$(NonStreamingPublishingMaxClients)' == ''">12</NonStreamingPublishingMaxClients>
       <StreamingPublishingMaxClients Condition="'$(UseStreamingPublishing)' == 'true' and '$(StreamingPublishingMaxClients)' == ''">16</StreamingPublishingMaxClients>
@@ -173,7 +172,6 @@
       FeedKeys="@(FeedKey)"
       FeedSasUris="@(FeedSasUri)"
       FeedOverrides="@(FeedOverride)"
-      ConvertPortablePdbsToWindowsPdbs="$(ConvertPortablePdbsToWindowsPdbs)"
       />
   </Target>
 

diff --git a/src/Microsoft.DotNet.Build.Tasks.Feed.Tests/PublishToSymbolServerTest.cs b/src/Microsoft.DotNet.Build.Tasks.Feed.Tests/PublishToSymbolServerTest.cs
@@ -91,7 +91,7 @@ public async Task TemporarySymbolDirectoryDoesNotExists()
             };
             var path = TestInputs.GetFullPath("Symbol");
             var buildAsset = new Dictionary<string, HashSet<Asset>>();
-            await task.HandleSymbolPublishingAsync(path, MsdlToken, SymWebToken, "", false, false, buildAsset, null, path);
+            await task.HandleSymbolPublishingAsync(path, MsdlToken, SymWebToken, "", false, buildAsset, null, path);
             Assert.True(task.Log.HasLoggedErrors);
         }
 

diff --git a/src/Microsoft.DotNet.Build.Tasks.Feed/src/PublishArtifactsInManifest.cs b/src/Microsoft.DotNet.Build.Tasks.Feed/src/PublishArtifactsInManifest.cs
@@ -139,8 +139,6 @@ public class PublishArtifactsInManifest : MSBuildTaskBase
         /// </summary>
         public bool PublishSpecialClrFiles { get; set; }
 
-        public bool ConvertPortablePdbsToWindowsPdbs { get; set; }
-
         /// <summary>
         /// If true, safety checks only print messages and do not error
         /// - Internal asset to public feed
@@ -344,7 +342,6 @@ internal PublishArtifactsInManifestBase ConstructPublishingV3Task(BuildModel bui
                 MsdlToken = this.MsdlToken,
                 SymbolPublishingExclusionsFile = this.SymbolPublishingExclusionsFile,
                 PublishSpecialClrFiles = this.PublishSpecialClrFiles,
-                ConvertPortablePdbsToWindowsPdbs = this.ConvertPortablePdbsToWindowsPdbs,
                 BuildQuality = this.BuildQuality,
                 ArtifactsBasePath =  this.ArtifactsBasePath,
                 AzdoApiToken = this.AzdoApiToken,

diff --git a/src/Microsoft.DotNet.Build.Tasks.Feed/src/PublishArtifactsInManifestBase.cs b/src/Microsoft.DotNet.Build.Tasks.Feed/src/PublishArtifactsInManifestBase.cs
@@ -396,20 +396,18 @@ public void CheckForStableAssetsInNonIsolatedFeeds()
         /// <param name="symbolPublishingExclusionsFile">Right now we do not add any files to this, so this is going to be null</param>
         /// <param name="publishSpecialClrFiles">If true, the special coreclr module indexed files like DBI, DAC and SOS are published</param>
         /// <param name="clientThrottle">To avoid starting too many processes</param>
-        /// <param name="convertPortablePdbsToWindowsPdbs">If true, convert protable pdbs to windows pdbs</param>
         /// <returns>Task</returns>
         public async Task PublishSymbolsUsingStreamingAsync(
             string pdbArtifactsBasePath,
             string msdlToken,
             string symWebToken,
             string symbolPublishingExclusionsFile,
             bool publishSpecialClrFiles,
-            bool convertPortablePdbsToWindowsPdbs,
             Dictionary<string, HashSet<Asset>> buildAssets,
             SemaphoreSlim clientThrottle)
         {
             Log.LogMessage(MessageImportance.High, 
-                $"Performing symbol publishing... \nExpirationInDays : {ExpirationInDays} \nConvertPortablePdbsToWindowsPdb : {convertPortablePdbsToWindowsPdbs} \ndryRun: false ");
+                $"Performing symbol publishing... \nExpirationInDays : {ExpirationInDays} \nConvertPortablePdbsToWindowsPdb : false \ndryRun: false ");
             var symbolCategory = TargetFeedContentType.Symbols;
 
             using HttpClient httpClient = CreateAzdoClient(AzureDevOpsOrg, false, AzureProject);
@@ -504,7 +502,7 @@ await PublishSymbolsHelper.PublishAsync(
                                         null,
                                         excludeFiles,
                                         ExpirationInDays,
-                                        convertPortablePdbsToWindowsPdbs,
+                                        false,
                                         publishSpecialClrFiles,
                                         null,
                                         false,
@@ -602,14 +600,12 @@ await PublishSymbolsHelper.PublishAsync(
         /// <param name="temporarySymbolsLocation">Path to Symbol.nupkgs</param>
         /// <param name="clientThrottle">To avoid starting too many processes</param>
         /// <param name="publishSpecialClrFiles">If true, the special coreclr module indexed files like DBI, DAC and SOS are published</param>
-        /// <param name="convertPortablePdbsToWindowsPdbs">If true, convert protable pdbs to windows pdbs</param>
         public async Task HandleSymbolPublishingAsync (
             string pdbArtifactsBasePath,
             string msdlToken, 
             string symWebToken,
             string symbolPublishingExclusionsFile,
             bool publishSpecialClrFiles,
-            bool convertPortablePdbsToWindowsPdbs,
             Dictionary<string, HashSet<Asset>> buildAssets,
             SemaphoreSlim clientThrottle = null,
             string temporarySymbolsLocation = null)
@@ -622,7 +618,6 @@ await PublishSymbolsUsingStreamingAsync(
                     symWebToken,
                     symbolPublishingExclusionsFile,
                     publishSpecialClrFiles,
-                    convertPortablePdbsToWindowsPdbs,
                     buildAssets,
                     clientThrottle);
             }
@@ -634,7 +629,6 @@ await PublishSymbolsfromBlobArtifactsAsync(
                     symWebToken,
                     symbolPublishingExclusionsFile,
                     publishSpecialClrFiles,
-                    convertPortablePdbsToWindowsPdbs,
                     temporarySymbolsLocation);
             }
         }
@@ -648,14 +642,12 @@ await PublishSymbolsfromBlobArtifactsAsync(
         /// <param name="symbolPublishingExclusionsFile">Right now we do not add any files to this, so this is going to be null</param>
         /// <param name="temporarySymbolsLocation">Path to Symbol.nupkgs</param>
         /// <param name="publishSpecialClrFiles">If true, the special coreclr module indexed files like DBI, DAC and SOS are published</param>
-        /// <param name="convertPortablePdbsToWindowsPdbs">If true, convert protable pdbs to windows pdbs</param>
         public async Task PublishSymbolsfromBlobArtifactsAsync(
             string pdbArtifactsBasePath,
             string msdlToken,
             string symWebToken,
             string symbolPublishingExclusionsFile,
             bool publishSpecialClrFiles,
-            bool convertPortablePdbsToWindowsPdbs,
             string temporarySymbolsLocation = null)
         {
             if (Directory.Exists(temporarySymbolsLocation))
@@ -690,7 +682,7 @@ public async Task PublishSymbolsfromBlobArtifactsAsync(
                     var serverPath = server.Key;
                     var token = server.Value;;
                     Log.LogMessage(MessageImportance.High,
-                        $"Performing symbol publishing...\nSymbolServerPath : ${serverPath} \nExpirationInDays : {ExpirationInDays} \nConvertPortablePdbsToWindowsPdb : {convertPortablePdbsToWindowsPdbs} \ndryRun: false \nTotal number of symbol files : {fileEntries.Length} ");
+                        $"Performing symbol publishing...\nSymbolServerPath : ${serverPath} \nExpirationInDays : {ExpirationInDays} \nConvertPortablePdbsToWindowsPdb : false \ndryRun: false \nTotal number of symbol files : {fileEntries.Length} ");
 
                     try
                     {
@@ -702,7 +694,7 @@ await PublishSymbolsHelper.PublishAsync(
                             filesToSymbolServer,
                             null,
                             ExpirationInDays,
-                            convertPortablePdbsToWindowsPdbs,
+                            false,
                             publishSpecialClrFiles,
                             null,
                             false,

diff --git a/src/Microsoft.DotNet.Build.Tasks.Feed/src/PublishArtifactsInManifestV3.cs b/src/Microsoft.DotNet.Build.Tasks.Feed/src/PublishArtifactsInManifestV3.cs
@@ -38,8 +38,6 @@ public class PublishArtifactsInManifestV3 : PublishArtifactsInManifestBase
 
         public bool AllowFeedOverrides { get; set; }
 
-        public bool ConvertPortablePdbsToWindowsPdbs { get; set;}
-
         public ITaskItem[] FeedKeys { get; set; }
         public ITaskItem[] FeedSasUris { get; set; }
 
@@ -195,7 +193,6 @@ await Task.WhenAll(new Task[]
                         SymWebToken,
                         SymbolPublishingExclusionsFile,
                         PublishSpecialClrFiles,
-                        ConvertPortablePdbsToWindowsPdbs,
                         buildAssets,
                         clientThrottle,
                         temporarySymbolsLocation)