Skip to content

Commit

Permalink
Fix VS Code Builds (#1475)
Browse files Browse the repository at this point in the history
The MIEngine_VSCode_Release pipeline was failing. This addresses it.

Fix was authored by Andrew Wang <[email protected]>
  • Loading branch information
gregg-miskelly authored Sep 27, 2024
1 parent 15173ad commit 40fdcd0
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
2 changes: 1 addition & 1 deletion eng/Scripts/CI-Build.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ param(
[Alias("c")]
[string]$Configuration="Debug",

[ValidateSet("win-x86", "win10-arm64")]
[ValidateSet("win-x86", "win-arm64")]
[Alias("r")]
[string]$RID="win-x86",

Expand Down
5 changes: 3 additions & 2 deletions eng/pipelines/steps/CopyAndPublishSymbols.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,14 @@
---
parameters:
OneESPT: false
SourceFolder: '$(Build.StagingDirectory)\drop'

steps:
- template: ../tasks/CopyFiles.yml
parameters:
displayName: 'Collect build symbols'
SourceFolder: '$(Build.StagingDirectory)\drop'
Contents: '$(Build.StagingDirectory)\drop\**\*.+(pdb|exe|dll)'
SourceFolder: ${{ parameters.SourceFolder }}
Contents: '${{ parameters.SourceFolder }}\**\*.+(pdb|exe|dll)'
TargetFolder: '$(Build.ArtifactStagingDirectory)/symbols'
CleanTargetFolder: true

Expand Down
3 changes: 2 additions & 1 deletion eng/pipelines/templates/VSCode-release.template.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
---
parameters:
rids: ["win-x86", "win10-arm64", "osx-x64", "osx-arm64", "linux-x64", "linux-arm", "linux-arm64", "linux-musl-x64", "linux-musl-arm64" ]
rids: ["win-x86", "win-arm64", "osx-x64", "osx-arm64", "linux-x64", "linux-arm", "linux-arm64", "linux-musl-x64", "linux-musl-arm64" ]

steps:
- checkout: self
Expand Down Expand Up @@ -28,6 +28,7 @@ steps:

- template: ../steps/CopyAndPublishSymbols.yml
parameters:
SourceFolder: '$(Build.StagingDirectory)\bin'
OneESPT: true

- script: |
Expand Down

0 comments on commit 40fdcd0

Please sign in to comment.