Skip to content

Commit

Permalink
Don't build with MKL by default. MKL can be a pain to install and I d…
Browse files Browse the repository at this point in the history
…on't want to break people who don't have it.
  • Loading branch information
fwilliams committed Nov 28, 2018
1 parent 44060c1 commit d25c75a
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,11 @@ def build_extension(self, ext):
with open("README.md", "r") as fh:
long_description = fh.read()

cmake_args = []
if 'USE_MKL' in os.environ or '--use-mkl' in sys.argv:
cmake_args.append('-DEIGEN_WITH_MKL=ON')
sys.argv.remove('--use-mkl')

setuptools.setup(
name="point-cloud-utils",
version="0.2.0",
Expand All @@ -85,7 +90,7 @@ def build_extension(self, ext):
"Programming Language :: Python :: 2.7",
"License :: OSI Approved :: GNU General Public License v3 (GPLv3)",
],
ext_modules=[CMakeExtension('point_cloud_utils', cmake_args=['-DEIGEN_WITH_MKL=ON'])],
ext_modules=[CMakeExtension('point_cloud_utils', cmake_args=cmake_args)],
cmdclass=dict(build_ext=CMakeBuild),
zip_safe=False,
install_requires=[
Expand Down

0 comments on commit d25c75a

Please sign in to comment.