Skip to content

Commit e33bc3d

Browse files
committed
build: separate vpack creation from vpack publication (#19380)
This will allow us to publish vpacks without making the build fail waiting for us to *merge* those vpacks into Windows. It also gives us better control over when and where the vpack update gets merged. (cherry picked from commit 6b42857) Service-Card-Id: PVTI_lADOAF3p4s4BBcTlzgfLAks Service-Version: 1.24
1 parent 6aae2d0 commit e33bc3d

File tree

3 files changed

+13
-5
lines changed

3 files changed

+13
-5
lines changed

build/config/GitCheckin.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"collection": "microsoft",
55
"project": "OS",
66
"repo": "os.2020",
7-
"name": "official/rs_we_adept_e4d2",
7+
"name": "official/ge_current_directwinpd_deep",
88
"workitem": "38106206",
99
"CheckinFiles": [
1010
{

build/pipelines/ob-release.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,8 +53,12 @@ parameters:
5353
displayName: "Publish Symbols to MSDL"
5454
type: boolean
5555
default: true
56+
- name: createVpack
57+
displayName: "Create a VPack for Windows"
58+
type: boolean
59+
default: false
5660
- name: publishVpackToWindows
57-
displayName: "Publish VPack to Windows"
61+
displayName: "Publish above VPack to Windows"
5862
type: boolean
5963
default: false
6064

@@ -89,6 +93,7 @@ extends:
8993
clientId: $(SigningOriginalClientId)
9094
terminalInternalPackageVersion: ${{ parameters.terminalInternalPackageVersion }}
9195
publishSymbolsToPublic: ${{ parameters.publishSymbolsToPublic }}
96+
createVpack: ${{ parameters.createVpack }}
9297
publishVpackToWindows: ${{ parameters.publishVpackToWindows }}
9398
symbolPublishingSubscription: $(SymbolPublishingServiceConnection)
9499
symbolPublishingProject: $(SymbolPublishingProject)

build/pipelines/templates-v2/pipeline-onebranch-full-release-build.yml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,9 @@ parameters:
4949
- name: symbolExpiryTime
5050
type: string
5151
default: 36530 # This is the default from PublishSymbols@2
52+
- name: createVpack
53+
type: boolean
54+
default: false
5255
- name: publishVpackToWindows
5356
type: boolean
5457
default: false
@@ -192,8 +195,8 @@ extends:
192195
ob_outputDirectory: $(JobOutputDirectory)
193196
ob_artifactBaseName: $(JobOutputArtifactName)
194197
### This job is also in charge of submitting the vpack to Windows if it's enabled
195-
ob_createvpack_enabled: ${{ and(parameters.buildTerminal, parameters.publishVpackToWindows) }}
196-
ob_updateOSManifest_enabled: ${{ and(parameters.buildTerminal, parameters.publishVpackToWindows) }}
198+
ob_createvpack_enabled: ${{ and(parameters.buildTerminal, parameters.createVpack) }}
199+
ob_updateOSManifest_enabled: ${{ and(parameters.buildTerminal, parameters.createVpack, parameters.publishVpackToWindows) }}
197200
### If enabled above, these options are in play.
198201
ob_createvpack_packagename: 'WindowsTerminal.app'
199202
ob_createvpack_owneralias: '[email protected]'
@@ -229,7 +232,7 @@ extends:
229232
New-Item "$(JobOutputDirectory)/vpack" -Type Directory
230233
displayName: Make sure the vpack directory exists
231234
232-
- ${{ if parameters.publishVpackToWindows }}:
235+
- ${{ if parameters.createVpack }}:
233236
- pwsh: |-
234237
Copy-Item -Verbose -Path "$(MsixBundlePath)" -Destination (Join-Path "$(JobOutputDirectory)/vpack" 'Microsoft.WindowsTerminal_8wekyb3d8bbwe.msixbundle')
235238
displayName: Stage msixbundle for vpack

0 commit comments

Comments
 (0)