-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Add package for ampl-asl C library #28198
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/ampl-asl/meta.yaml:
|
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 ( |
29f2553
to
fc66ab9
Compare
@conda-forge/staged-recipes @conda-forge/help-c-cpp @conda-forge/ipopt @conda-forge/ampl-mp the PR is ready for review. |
To help direct your pull request to the best reviewers, please mention a topic-specifc team if your recipe matches any of the following: conda-forge/help-c-cpp, conda-forge/help-cdts, conda-forge/help-go, conda-forge/help-java, conda-forge/help-julia, conda-forge/help-nodejs, conda-forge/help-perl, conda-forge/help-python, conda-forge/help-python-c, conda-forge/help-r, conda-forge/help-ruby,or conda-forge/help-rust. Thanks! |
Thanks! |
The PR adds a recipe for the
ampl-asl
library (repology page: https://repology.org/project/ampl-asl/). The naming of the recipe and the package mirrors the one of the related packageampl-mp
, maintained at https://github.com/conda-forge/ampl-mp-feedstock .This package is now added as ampl-mp 4.0.0 is not vendoring anymore ampl-asl, so we need to package it as separate project, see:
Furthermore, as ipopt actually just depends on asl, by packaging ampl-asl we can just depend on ampl-asl rather then depend on ampl-mp that also includes a public ampl-asl vendored library.
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).