-
Notifications
You must be signed in to change notification settings - Fork 547
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
[FFLAS_FFPACK] New recipe #9661
base: master
Are you sure you want to change the base?
[FFLAS_FFPACK] New recipe #9661
Conversation
…FFPACK to FFLAS_FFPACK
Hi @giordano , this is a dependency that I created out of my original attempt to have a binary package for the library Spasm. In particular, I'm working on a recipe for Spasm with a Dependency("FFLAS_FFPACK_jll"; compat="2.5.0") in it. For now, of course, this recipe fails because FFLAS_FFPACK is not yet merged; however, is there any way I can test my new recipe, to make sure that I cooked FFLAS_FFPACK correctly? |
# The products that we will ensure are always built | ||
products = [ | ||
FileProduct("include/fflas-ffpack/fflas-ffpack-config.h", :fflas_ffpack_config_h) | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the product only a header file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, this is almost-pure C++ library, which produces lots of files, all in include/ except for a bin/fflas-ffpack-config. Should I include more?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After opening the archive and not seeing any binary product I believe the correct answer should have been: yes, this is a header-only package.
F/FFLAS_FFPACK/bundled/patches/fflas-ffpack-config-without-run.patch
Outdated
Show resolved
Hide resolved
Dear @giordano , I believe I addressed all the points you raised. Is the correct procedure to wait for the formula to be merged, and hope that it works as expected for Spasm? |
This is a recipe for the FFLAS_FFPACK - Finite Field Linear Algebra Subroutines package.
It is based on Givaro (see Givaro_jll), and is part of the LinBox series of libraries. It is purely C++.