OnnxRuntime nuget package uses build and not buildTransitive

[ericstj]

Describe the issue

The microsoft.ml.onnxruntime.managed package includes targets to warn the user when referencing on an unsupported architecture. Those targets will not be applied when the package is referenced transitively because they are placed in build and not buildTransitive.

onnxruntime/csharp/src/Microsoft.ML.OnnxRuntime/Microsoft.ML.OnnxRuntime.csproj

Lines 294 to 298 in c20ced4

	<None Include="targets\netstandard\$(PackageId).targets"
	PackagePath="build\netstandard2.0\$(PackageId).targets"
	Pack="true"
	Visible="false"
	/>

Probably those should be copied to buildTransitive https://learn.microsoft.com/en-us/nuget/consume-packages/package-references-in-project-files#controlling-dependency-assets

Here's a similar issue I noticed with LightGBM: microsoft/LightGBM#6344

To reproduce

See attached project
onnxRuntime-buildTransitiveSample.zip

Note that the targets are not included when running in the app - which specifies a platformtarget of Itanium and should trigger the Error in those targtes.

Urgency

Probably not too urgent since OnnxRuntime supports most architectures and the warning won't likely be hit, but I wanted to raise this since I noticed the problem.

Platform

Windows

OS Version

11 22H2

ONNX Runtime Installation

Built from Source

ONNX Runtime Version or Commit ID

1.17.1

ONNX Runtime API

C#

Architecture

X64

Execution Provider

Default CPU

Execution Provider Library Version

No response

[github-actions]

This issue has been automatically marked as stale due to inactivity and will be closed in 30 days if no further activity occurs. If further support is needed, please provide an update and/or more details.