Use build_ext instead of install/develop, fix rpath for Linux

Author: gormster

setup.py

@@ -1,8 +1,7 @@
 import os
 import platform
 from setuptools import setup
-from setuptools.command.develop import develop
-from setuptools.command.install import install
+from setuptools.command.build_ext import build_ext
 
 # "import" __version__
 __version__ = 'unknown'
@@ -38,6 +37,16 @@
     package_data = None
     zip_safe = True
 
+
+def post_install():
+    if system == 'Darwin':
+        # Bizarrely, on macOS, there doesn't seem to be another way to do this: you MUST use install_name_tool
+        import subprocess
+        # It could be either of these, so try both
+        subprocess.run(["install_name_tool", "-change", "/usr/local/lib/libportaudio.dylib", "@rpath/libportaudio.dylib", "./_sounddevice.abi3.so"])
+        subprocess.run(["install_name_tool", "-change", "/usr/local/lib/libportaudio.2.dylib", "@rpath/libportaudio.dylib", "./_sounddevice.abi3.so"])
+
+
 try:
     from wheel.bdist_wheel import bdist_wheel
 except ImportError:
@@ -60,32 +69,16 @@ def get_tag(self):
 
     cmdclass = {'bdist_wheel': bdist_wheel_half_pure}
 
-def post_install():
-    if system == 'Darwin':
-        # Bizarrely, on macOS, there doesn't seem to be another way to do this: you MUST use install_name_tool
-        import subprocess
-        # It could be either of these, so try both
-        subprocess.run(["install_name_tool", "-change", "/usr/local/lib/libportaudio.dylib", "@rpath/libportaudio.dylib", "./_sounddevice.abi3.so"])
-        subprocess.run(["install_name_tool", "-change", "/usr/local/lib/libportaudio.2.dylib", "@rpath/libportaudio.dylib", "./_sounddevice.abi3.so"])
 
-class PostDevelopCommand(develop):
+class PostBuildCommand(build_ext):
     """Post-installation for development mode."""
 
     def run(self):
-        develop.run(self)
-        post_install()
-
-
-class PostInstallCommand(install):
-    """Post-installation for installation mode."""
-
-    def run(self):
-        install.run(self)
+        build_ext.run(self)
         post_install()
 
 
-cmdclass['develop'] = PostDevelopCommand
-cmdclass['install'] = PostInstallCommand
+cmdclass['build_ext'] = PostBuildCommand
 
 
 setup(

sounddevice_build.py

@@ -3,17 +3,26 @@
 import platform as _platform
 
 extra_header_files = ""
+extra_link_args = []
 
 if _platform.system() == 'Darwin':
     extra_header_files = """
     #include "pa_mac_core.h"
     """
+    extra_link_args = ["-Wl,-rpath,@loader_path/_sounddevice_data/portaudio-binaries"]
 elif _platform.system() == 'Windows':
     extra_header_files = """
     #include "pa_win_waveformat.h"
     #include "pa_asio.h"
     #include "pa_win_wasapi.h"
     """
+    # TODO: how to do RPATH on Windows?
+elif _platform.system() == 'Linux':
+    extra_header_files = """
+    #include "pa_linux_alsa.h"
+    """
+    extra_link_args = ["-Wl,-rpath,$ORIGIN/_sounddevice_data/portaudio-binaries"]
+
 
 ffibuilder = FFI()
 ffibuilder.set_source(
@@ -26,7 +35,7 @@
     include_dirs=["./include"],
     libraries=["portaudio"],
     library_dirs=["./_sounddevice_data/portaudio-binaries"],
-    extra_link_args=["-Wl,-rpath,@loader_path/_sounddevice_data/portaudio-binaries"]
+    extra_link_args=extra_link_args
 )
 
 ffibuilder.cdef("""