Remove symbol stripping from pipeline scripts (#33911)
authorAdeel Mujahid <adeelbm@outlook.com>
Wed, 25 Mar 2020 14:11:16 +0000 (16:11 +0200)
committerGitHub <noreply@github.com>
Wed, 25 Mar 2020 14:11:16 +0000 (15:11 +0100)
Symbols stripping is unconditionally on for native builds.

eng/pipelines/coreclr/templates/build-job.yml
eng/pipelines/libraries/base-job.yml
eng/pipelines/mono/templates/build-job.yml

index 2a58376..ac283f7 100644 (file)
@@ -54,12 +54,6 @@ jobs:
       value: ${{ parameters.osGroup }}
     - name: osSubgroup
       value: ${{ parameters.osSubgroup }}
-    - name: stripSymbolsArg
-      value: ''
-    # Strip symbols only on the release build
-    - ${{ if eq(parameters.buildConfig, 'Release') }}:
-      - name: stripSymbolsArg
-        value: '-stripsymbols'
     - name: compilerArg
       value: ''
     - name: publishLogsArtifactPrefix
@@ -132,7 +126,7 @@ jobs:
 
     # Build CoreCLR Runtime
     - ${{ if ne(parameters.osGroup, 'Windows_NT') }}:
-      - script: $(coreClrRepoRootDir)build-runtime$(scriptExt) $(buildConfig) $(archType) $(crossArg) -ci $(compilerArg) $(stripSymbolsArg) $(officialBuildIdArg)
+      - script: $(coreClrRepoRootDir)build-runtime$(scriptExt) $(buildConfig) $(archType) $(crossArg) -ci $(compilerArg) $(officialBuildIdArg)
         displayName: Build CoreCLR Runtime
     - ${{ if eq(parameters.osGroup, 'Windows_NT') }}:
       - script: set __TestIntermediateDir=int&&$(coreClrRepoRootDir)build-runtime$(scriptExt) $(buildConfig) $(archType) -ci $(enforcePgoArg) $(officialBuildIdArg)
index e24d0d3..c39cdd4 100644 (file)
@@ -41,7 +41,6 @@ jobs:
 
         - _msbuildCommonParameters: ''
         # rename this variable, due to collision with build-native.proj
-        - _stripSymbolsArgYaml: ''
         - _runtimeOSArg: ''
         - _finalFrameworkArg: ''
         - _buildScript: $(_buildScriptFileName)$(scriptExt)
@@ -111,10 +110,8 @@ jobs:
         # Non-Windows variables
         - ${{ if ne(parameters.osGroup, 'Windows_NT') }}:
           - _buildScript: ./$(_buildScriptFileName)$(scriptExt)
-          - ${{ if eq(parameters.isOfficialBuild, 'true') }}:
-            - _stripSymbolsArgYaml: -stripSymbols
 
-        - _buildArguments: -configuration ${{ parameters.buildConfig }} -ci -arch ${{ parameters.archType }} $(_finalFrameworkArg) $(_stripSymbolsArgYaml) $(_testScopeArg) $(_warnAsErrorArg) $(_runtimeOSArg) $(_msbuildCommonParameters) $(_runtimeArtifactsPathArg) $(_crossBuildPropertyArg)
+        - _buildArguments: -configuration ${{ parameters.buildConfig }} -ci -arch ${{ parameters.archType }} $(_finalFrameworkArg) $(_testScopeArg) $(_warnAsErrorArg) $(_runtimeOSArg) $(_msbuildCommonParameters) $(_runtimeArtifactsPathArg) $(_crossBuildPropertyArg)
         - ${{ parameters.variables }}
 
       dependsOn:
index 6b3f927..878f34e 100644 (file)
@@ -55,10 +55,6 @@ jobs:
       value: ''
     - name: osOverride
       value: ''
-    # Strip symbols only on the release build
-    - ${{ if eq(parameters.buildConfig, 'Release') }}:
-      - name: stripSymbolsArg
-        value: '-stripsymbols'
     - ${{ if and(eq(variables['System.TeamProject'], 'internal'), ne(variables['Build.Reason'], 'PullRequest')) }}:
       - name: officialBuildIdArg
         value: '/p:officialBuildId=$(Build.BuildNumber)'