Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove 10.14 suffix from osx nuget package #865

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jeffpardy
Copy link

Updates the specified runtime for osx to remove the OS specific RID. This is no longer supported since .NET 8 and generates the following warning as well as runtime errors.

Matches the same changes made in microsoft/onnxruntime with microsoft/onnxruntime#17277

Build Warning

/usr/local/share/dotnet/sdk/9.0.101/Sdks/Microsoft.NET.Sdk/targets/Microsoft.NET.Sdk.targets(322,5): warning NETSDK1206: Found version-specific or distribution-specific runtime identifier(s): osx.10.14-arm64, osx.10.14-x64. Affected libraries: Microsoft.ML.OnnxRuntime.Extensions. In .NET 8.0 and higher, assets for version-specific and distribution-specific runtime identifiers will not be found by default. See https://aka.ms/dotnet/rid-usage for details.

Runtime Error

Error Message
      : Microsoft.ML.OnnxRuntime.OnnxRuntimeException : [ErrorCode:NoSuchFile] The ONNX Runtime extensions library
       was not found. The Microsoft.ML.OnnxRuntime.Extensions NuGet package must be referenced by the project to u
      se 'OrtExtensions.RegisterCustomOps.
      Stack Trace:
         at Microsoft.ML.OnnxRuntime.SessionOptions.RegisterOrtExtensions()

@jeffpardy jeffpardy requested a review from a team as a code owner December 22, 2024 01:03
@jeffpardy
Copy link
Author

Thanks @wenbingl!
@skottmckay @edgchen1 anything else I can do?

@edgchen1
Copy link
Contributor

edgchen1 commented Jan 6, 2025

@jeffpardy The changes look fine to me. I'm not very familiar with the NuGet setup though. Were you able to verify that this fixes the issues you encountered?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants