Skip to content

Commit 7c7a3d2

Browse files
authored
Merge pull request #29182 from dotnet/marcpopMSFT-improvepublish
Improve build stability by ensuring we aren't double publishing everywhere
2 parents 27f5eca + bdeec45 commit 7c7a3d2

File tree

4 files changed

+16
-8
lines changed

4 files changed

+16
-8
lines changed

.vsts-ci-richnav.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,12 @@ stages:
5252
- _SignArgs: ''
5353
- _InternalRuntimeDownloadArgs: ''
5454
steps:
55-
- script: eng\CIBuild.cmd
55+
- powershell: eng\common\build.ps1
56+
-restore
57+
-ci
58+
-build
59+
-pack
60+
-sign
5661
-configuration $(_BuildConfig)
5762
$(_PublishArgs)
5863
$(_SignArgs)

.vsts-ci.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@ variables:
1414
value: Roslyn-Project-System
1515
- name: _DotNetPublishToBlobFeed
1616
value: false
17+
- name: _CIBuild
18+
value: -restore -build -sign -pack -ci
1719
- name: _DotNetArtifactsCategory
1820
value: .NETCore
1921
- name: _DotNetValidationArtifactsCategory

eng/CIBuild.cmd

Lines changed: 0 additions & 2 deletions
This file was deleted.

eng/build.yml

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,8 @@ jobs:
6565
env:
6666
Token: $(dn-bot-dnceng-artifact-feeds-rw)
6767
- ${{ if eq(parameters.agentOs, 'Windows_NT') }}:
68-
- script: eng\CIBuild.cmd
68+
- powershell: eng\common\build.ps1
69+
$(_CIBuild)
6970
-configuration $(_BuildConfig)
7071
$(_PublishArgs)
7172
$(_SignArgs)
@@ -90,7 +91,8 @@ jobs:
9091
HelixAccessToken: $(_HelixApiToken)
9192

9293
- ${{ if eq(parameters.agentOs, 'Windows_NT_FullFramework') }}:
93-
- script: eng\CIBuild.cmd
94+
- powershell: eng\common\build.ps1
95+
$(_CIBuild)
9496
-configuration $(_BuildConfig)
9597
$(_SignArgs)
9698
$(_OfficialBuildIdArgs)
@@ -116,19 +118,20 @@ jobs:
116118
HelixAccessToken: $(_HelixApiToken)
117119

118120
- ${{ if eq(parameters.agentOs, 'Windows_NT_TestAsTools') }}:
119-
- script: eng\CIBuild.cmd
121+
- powershell: eng\common\build.ps1
122+
$(_CIBuild)
120123
-configuration $(_BuildConfig)
121124
$(_SignArgs)
122125
$(_OfficialBuildIdArgs)
123-
$(_Test)
124126
$(_InternalRuntimeDownloadArgs)
125127
/p:RunTestsAsTool=true
126128
displayName: Build
127129
env:
128130
BuildConfig: $(_BuildConfig)
129131

130132
- ${{ if notIn(parameters.agentOs, 'Windows_NT', 'Windows_NT_FullFramework', 'Windows_NT_TestAsTools') }}:
131-
- script: eng/common/cibuild.sh
133+
- script: eng/common/build.sh
134+
$(_CIBuild)
132135
--configuration $(_BuildConfig)
133136
$(_SignArgs)
134137
$(_OfficialBuildIdArgs)

0 commit comments

Comments
 (0)