Paul Wise

Results 252 comments of Paul Wise

Please note that these variables should be set as early as possible in the main cmake file, so that they apply to all of the targets.

Looks like there is also the [FindPython](https://cmake.org/cmake/help/latest/module/FindPython.html) cmake module, which abstracts away the Python version, including adding a `Python_add_library` function. I suggest dropping the custom Python version handling and using...

Actually, it looks like `Python_add_library` doesn't do any installing and the files just need to be installed to the `Python_SITELIB` (for .py files) and `Python_SITEARCH` (for .so files) directories.

In order to get the `Python_SITELIB` and `Python_SITEARCH` variables, you need the `Interpreter` component in the call to `find_package(Python)` and `find_package(Python3)`.

If someone were to fix up and merge #136 that fixes #134 requesting support for 64-bit ARM, then this issue would be fixed too.

Fixing this issue will also fix #316 too if SIMDe is used to fix it.

@mcelhennyi I noticed that there are conflicts with the latest commit of the master branch, would you mind rebasing it? I'm considering adding it to the Debian package of SPTAG.

Merging this issue will also fix #316 too.

I tried to create a new patch but SPTAG now uses AVX512 and SIMDe is missing three functions and is missing three native aliases (useful to avoid having to change...

Debian source packages are not exactly proper pristine clean source only trees. Even Debian trees aren't proper pristine clean source only trees. The same applies to other distros and upstream...