Skip to content

Commit b1174a6

Browse files
committed
If the users selects a language, let's have it actually choose the language they selected. (Yaml vs yaml) (#288153)
Fixes #287005
1 parent cce0642 commit b1174a6

File tree

3 files changed

+38
-12
lines changed

3 files changed

+38
-12
lines changed

build/azure-pipelines/darwin/codesign.ts

Lines changed: 35 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -10,19 +10,44 @@ async function main() {
1010
const arch = e('VSCODE_ARCH');
1111
const esrpCliDLLPath = e('EsrpCliDllPath');
1212
const pipelineWorkspace = e('PIPELINE_WORKSPACE');
13+
const buildSourcesDirectory = e('BUILD_SOURCESDIRECTORY');
1314

14-
const folder = `${pipelineWorkspace}/vscode_client_darwin_${arch}_archive`;
15-
const glob = `VSCode-darwin-${arch}.zip`;
15+
const clientFolder = `${pipelineWorkspace}/vscode_client_darwin_${arch}_archive`;
16+
const clientGlob = `VSCode-darwin-${arch}.zip`;
1617

17-
// Codesign
18-
printBanner('Codesign');
19-
const codeSignTask = spawnCodesignProcess(esrpCliDLLPath, 'sign-darwin', folder, glob);
20-
await streamProcessOutputAndCheckResult('Codesign', codeSignTask);
18+
const serverFolder = `${buildSourcesDirectory}/.build/darwin/server`;
19+
const serverGlob = `vscode-server-darwin-${arch}.zip`;
20+
const webGlob = `vscode-server-darwin-${arch}-web.zip`;
2121

22-
// Notarize
23-
printBanner('Notarize');
24-
const notarizeTask = spawnCodesignProcess(esrpCliDLLPath, 'notarize-darwin', folder, glob);
25-
await streamProcessOutputAndCheckResult('Notarize', notarizeTask);
22+
// Start codesign processes in parallel
23+
const codeSignClientTask = spawnCodesignProcess(esrpCliDLLPath, 'sign-darwin', clientFolder, clientGlob);
24+
const codeSignServerTask = spawnCodesignProcess(esrpCliDLLPath, 'sign-darwin', serverFolder, serverGlob);
25+
const codeSignWebTask = spawnCodesignProcess(esrpCliDLLPath, 'sign-darwin', serverFolder, webGlob);
26+
27+
// Await codesign results
28+
printBanner('Codesign client');
29+
await streamProcessOutputAndCheckResult('Codesign client', codeSignClientTask);
30+
31+
printBanner('Codesign server');
32+
await streamProcessOutputAndCheckResult('Codesign server', codeSignServerTask);
33+
34+
printBanner('Codesign web');
35+
await streamProcessOutputAndCheckResult('Codesign web', codeSignWebTask);
36+
37+
// Start notarize processes in parallel (after codesigning is complete)
38+
const notarizeClientTask = spawnCodesignProcess(esrpCliDLLPath, 'notarize-darwin', clientFolder, clientGlob);
39+
const notarizeServerTask = spawnCodesignProcess(esrpCliDLLPath, 'notarize-darwin', serverFolder, serverGlob);
40+
const notarizeWebTask = spawnCodesignProcess(esrpCliDLLPath, 'notarize-darwin', serverFolder, webGlob);
41+
42+
// Await notarize results
43+
printBanner('Notarize client');
44+
await streamProcessOutputAndCheckResult('Notarize client', notarizeClientTask);
45+
46+
printBanner('Notarize server');
47+
await streamProcessOutputAndCheckResult('Notarize server', notarizeServerTask);
48+
49+
printBanner('Notarize web');
50+
await streamProcessOutputAndCheckResult('Notarize web', notarizeWebTask);
2651
}
2752

2853
main().then(() => {

build/azure-pipelines/darwin/product-build-darwin.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,14 +57,14 @@ jobs:
5757
sbomPackageVersion: $(Build.SourceVersion)
5858
- output: pipelineArtifact
5959
targetPath: $(Build.ArtifactStagingDirectory)/out/vscode_server_darwin_$(VSCODE_ARCH)_archive/vscode-server-darwin-$(VSCODE_ARCH).zip
60-
artifactName: vscode_server_darwin_$(VSCODE_ARCH)_archive-unsigned
60+
artifactName: vscode_server_darwin_$(VSCODE_ARCH)_archive
6161
displayName: Publish server archive
6262
sbomBuildDropPath: $(Agent.BuildDirectory)/vscode-server-darwin-$(VSCODE_ARCH)
6363
sbomPackageName: "VS Code macOS $(VSCODE_ARCH) Server"
6464
sbomPackageVersion: $(Build.SourceVersion)
6565
- output: pipelineArtifact
6666
targetPath: $(Build.ArtifactStagingDirectory)/out/vscode_web_darwin_$(VSCODE_ARCH)_archive/vscode-server-darwin-$(VSCODE_ARCH)-web.zip
67-
artifactName: vscode_web_darwin_$(VSCODE_ARCH)_archive-unsigned
67+
artifactName: vscode_web_darwin_$(VSCODE_ARCH)_archive
6868
displayName: Publish web server archive
6969
sbomBuildDropPath: $(Agent.BuildDirectory)/vscode-server-darwin-$(VSCODE_ARCH)-web
7070
sbomPackageName: "VS Code macOS $(VSCODE_ARCH) Web"

build/azure-pipelines/darwin/steps/product-build-darwin-compile.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -261,6 +261,7 @@ steps:
261261
env:
262262
EsrpCliDllPath: $(EsrpCliDllPath)
263263
SYSTEM_ACCESSTOKEN: $(System.AccessToken)
264+
BUILD_SOURCESDIRECTORY: $(Build.SourcesDirectory)
264265
displayName: ✍️ Codesign & Notarize
265266

266267
- ${{ if or(eq(parameters.VSCODE_RUN_ELECTRON_TESTS, true), eq(parameters.VSCODE_RUN_BROWSER_TESTS, true), eq(parameters.VSCODE_RUN_REMOTE_TESTS, true)) }}:

0 commit comments

Comments
 (0)