-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
recipe for the ninja python module #24850
Conversation
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/ninja-python:
|
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 ( |
… into ninja-python
@conda-forge/help-python, ready for review! |
There's no reason to re-papckage a ninja when there's already a ninja package in conda-forge. Maybe we can use the same approach as NixOS/nixpkgs#246963 |
@isuruf thanks for reaching out! Apologies, I'm not following what the suggested path forward is. My understanding is ninja-python-distributions allows for python integration of ninja. Is your approach to update FWIW, I'm unfamiliar with ninja, and trying to resolve downstream dependencies for another build which cannot |
@isuruf and/or @conda-forge/help-python any thoughts on the above? |
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).