fix build warnings (#2355)

* fix build warnings

* cython fixes

* cleanup transformations build

* little more
old-commit-hash: 96b637737bd42b334bffb188d278766bd0abe6c6
This commit is contained in:
Adeeb Shihadeh
2020-10-17 12:40:01 -07:00
committed by GitHub
parent dae628002f
commit f92c30b164
11 changed files with 29 additions and 44 deletions
+10 -32
View File
@@ -1,42 +1,20 @@
import os
import numpy
import sysconfig
from Cython.Build import cythonize
from Cython.Distutils import build_ext
from distutils.core import Extension, setup # pylint: disable=import-error,no-name-in-module
def get_ext_filename_without_platform_suffix(filename):
name, ext = os.path.splitext(filename)
ext_suffix = sysconfig.get_config_var('EXT_SUFFIX')
if ext_suffix == ext:
return filename
ext_suffix = ext_suffix.replace(ext, '')
idx = name.find(ext_suffix)
if idx == -1:
return filename
else:
return name[:idx] + ext
class BuildExtWithoutPlatformSuffix(build_ext):
def get_ext_filename(self, ext_name):
filename = super().get_ext_filename(ext_name)
return get_ext_filename_without_platform_suffix(filename)
from common.cython_hacks import BuildExtWithoutPlatformSuffix
setup(
name='Cython transformations wrapper',
cmdclass={'build_ext': BuildExtWithoutPlatformSuffix},
ext_modules=cythonize(
Extension(
"transformations",
sources=["transformations.pyx"],
language="c++",
extra_compile_args=["-std=c++14", "-Wno-nullability-completeness"],
include_dirs=[numpy.get_include()],
Extension(
"transformations",
sources=["transformations.pyx"],
language="c++",
extra_compile_args=["-std=c++14", "-Wno-cpp"],
include_dirs=[numpy.get_include()],
),
nthreads=4,
)
))
)