From 1e8ff8684d9356d37d495012fe3e00a5af0cd775 Mon Sep 17 00:00:00 2001 From: Kumi Date: Sat, 10 Feb 2024 20:00:27 +0100 Subject: [PATCH] Updated CXX env var and removed redundant library_dirs in setup Enhanced the 'CXX' environment variable assignment in setup.py to include the '-shlib' flag for shared library support. Also, streamlined the Extension configuration by eliminating the now unnecessary specification of default library directories, simplifying the build process. Refactors setup configuration for efficiency and compatibility with shared libraries. --- setup.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/setup.py b/setup.py index dbd5afb..18cb936 100644 --- a/setup.py +++ b/setup.py @@ -6,14 +6,13 @@ import os from Cython.Build import cythonize -os.environ['CXX'] = 'h5c++' +os.environ['CXX'] = 'h5c++ -shlib' ext_modules = [ Extension( 'limedriver.binding', sources=['src/limedriver/limedriver.pyx', 'extern/limedriver/src/limedriver.cpp'], include_dirs=["extern/limedriver/src/", "/usr/include/hdf5/serial/"], # TODO: This is REALLY ugly. - library_dirs=["/usr/lib/"], libraries=["LimeSuite", "hdf5_cpp", "hdf5"], language="c++", ),