-
Notifications
You must be signed in to change notification settings - Fork 10
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
Added support of building using the deps fetched from pkg-config. #1
base: master
Are you sure you want to change the base?
Conversation
Hi @KOLANICH thank you. A couple of questions/notes:
|
It works, the test has returned no error. BTW, the test itself is done wrong. One needs to use
It would require changes in the original library code. The dir with the headers has incorrect layout.
It is a kind of a standard. Just text files of a certain format in specifjc location. Can be distributed by any package manager and even without them, but to distribute them one has to generate them. Their main use is on nix systems, but they can also be used on other systems, if there is a preaggreed location where The original library CMake scripts generate them and install them (but I don't recommend to use BTW, it may be possible to get rid of a cext and use ctypes. It would allow using the lib in other implementations of python without recompilaion, and also would elimjnate any need in compilation of the cext, so users without compilers in their system (Visual Studio is shit, though for lot of packages MinGW-w64 works fine even on pythons compiled with VS) would be able to install the lib. |
Ohh, there's a fastpfor header link, so it's ok. Regarding pkgconfig: how does it interact with pip install? The library isn't particular popular, so I wouldn't bother about shipping a windows version. |
|
Ok, I will need to come back to this a bit later. BTW, I am not really eager to support Windows for this project unless it (for some reason) becomes popular. |
No description provided.