-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
add kmsxx recipe #30713
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
base: main
Are you sure you want to change the base?
add kmsxx recipe #30713
Conversation
Hi! This is the staged-recipes linter and your PR looks excellent! 🚀 |
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 ( Here's what I've got... For recipes/kmsxx/meta.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/16723881274. Examine the logs at this URL for more detail. |
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 ( |
@traversaro before I ping help-c-cpp do you have any suggestion on this? The error is a missing linux header AFAIU:
By the way, this is a missing piece towards building |
Sure! Sorry I am currently a bit far from keyboard so I may be slow to reply. Based on https://conda-metadata-app.streamlit.app/Search_by_file_path, I guess that is a Linux kernel header, so you can add a dependency on the kernel_headers or sysroot CDT, I do not remember exactly. |
1 similar comment
Sure! Sorry I am currently a bit far from keyboard so I may be slow to reply. Based on https://conda-metadata-app.streamlit.app/Search_by_file_path, I guess that is a Linux kernel header, so you can add a dependency on the kernel_headers or sysroot CDT, I do not remember exactly. |
Thanks! I was expecting something like that but didn't know where to look. That web tool is fantastic, didn't know about it. Still, that header cannot be found. I'm thinking that it might be a problem with the include path.. |
Probably the correct dependency is the sysroot one that also sets the include path? |
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 ( Here's what I've got... For recipes/kmsxx/meta.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/16771759700. Examine the logs at this URL for more detail. |
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 ( |
Well, we moved forward 🥳 (thanks @traversaro!), and now I'm stuck here:
|
That sounds like a shared library not being installed? If fbtest is not actually useful, perhaps we can just delete it from the install. |
@traversaro I would be grateful if you could review the recipe as well as @conda-forge/help-c-cpp Thank you! |
Hi! This is the staged-recipes linter and your PR looks excellent but I have some suggestions. File-specific lints and/or hints:
|
Hi! This is the staged-recipes linter and your PR looks excellent! 🚀 |
https://github.com/tomba/kmsxx
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).