Install C++ headers and shared library

conda-forge / onnxruntime-feedstock

A conda-smithy repository for onnxruntime.

BSD 3-Clause "New" or "Revised" License

1 stars 19 forks source link

Install C++ headers and shared library #34

Closed ayllon closed 2 years ago

ayllon commented 2 years ago

Hello,

This patch just enables and install the C++ headers and shared libraries, so onnxruntime can be also be used from C++ applications like sourcextractor++ (which is not yet part of conda-forge, but may be one day)

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[X] Re-rendered with the latest conda-smithy (Use the phrase code>@<space/conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

conda-forge-linter commented 2 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

xhochy commented 2 years ago

This should go into a separate output so that we have a package that isn't dependent on a Python version.

conda-forge-linter commented 2 years ago

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.

Here's what I've got...

For recipe:

The top level meta keys are in an unexpected order. Expecting ['package', 'source', 'build', 'requirements', 'test', 'outputs', 'about', 'extra'].
The outputs section contained an unexpected subsection name. run is not a valid subsection name.

ayllon commented 2 years ago

This should go into a separate output so that we have a package that isn't dependent on a Python version.

Thanks for the feedback. I have added it, but just saw the linter gave some warnings. I will fix them.

conda-forge-linter commented 2 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

ayllon commented 2 years ago

Hello,

I have rebased on top of main and added a simple test for the onnxruntime-cpp package

ayllon commented 2 years ago

You'll need to rebase one more time to get to version 1.12.0.

Done!