From 897c1c1f0570ad445b0e3ef8316aad63ffff7daf Mon Sep 17 00:00:00 2001 From: Scott McKay Date: Tue, 14 Nov 2023 14:01:59 +1000 Subject: [PATCH] Set DML package name correctly in CI (#18405) ### Description Set DML package name correctly so the build doesn't try and include mobile targets. ### Motivation and Context Fix packaging pipeline. --- .../azure-pipelines/c-api-noopenmp-packaging-pipelines.yml | 1 - .../github/azure-pipelines/nuget/templates/dml-vs-2022.yml | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/tools/ci_build/github/azure-pipelines/c-api-noopenmp-packaging-pipelines.yml b/tools/ci_build/github/azure-pipelines/c-api-noopenmp-packaging-pipelines.yml index ae21581885..0eccd71e47 100644 --- a/tools/ci_build/github/azure-pipelines/c-api-noopenmp-packaging-pipelines.yml +++ b/tools/ci_build/github/azure-pipelines/c-api-noopenmp-packaging-pipelines.yml @@ -1094,7 +1094,6 @@ stages: DoNugetPack : 'true' DoCompliance: 'false' DoEsrp: ${{ parameters.DoEsrp }} - OrtPackageId: 'Microsoft.ML.OnnxRuntime.DirectML' NuPackScript: | msbuild $(Build.SourcesDirectory)\csharp\OnnxRuntime.CSharp.proj /p:Configuration=RelWithDebInfo /t:CreatePackage /p:OrtPackageId=Microsoft.ML.OnnxRuntime.DirectML /p:IsReleaseBuild=${{ parameters.IsReleaseBuild }} copy $(Build.SourcesDirectory)\csharp\src\Microsoft.ML.OnnxRuntime\bin\RelWithDebInfo\*.nupkg $(Build.ArtifactStagingDirectory) diff --git a/tools/ci_build/github/azure-pipelines/nuget/templates/dml-vs-2022.yml b/tools/ci_build/github/azure-pipelines/nuget/templates/dml-vs-2022.yml index 2d92108efb..4e7093f04a 100644 --- a/tools/ci_build/github/azure-pipelines/nuget/templates/dml-vs-2022.yml +++ b/tools/ci_build/github/azure-pipelines/nuget/templates/dml-vs-2022.yml @@ -15,7 +15,7 @@ parameters: EnvSetupScript: 'setup_env.bat' AgentPool: 'onnxruntime-Win-CPU-2022' AgentDemands: [] - OrtPackageId: Microsoft.ML.OnnxRuntime + OrtPackageId: Microsoft.ML.OnnxRuntime.DirectML BuildConfigurations: ['RelWithDebInfo'] # Options: Debug, RelWithDebInfo RunTests : 'true' EnableLto: true