Merge pull request #11398 from iblis17/gnu-inline

BLD: add CFLAGS `-fgnu89-inline` on FreeBSD 10+

Author: jreback

doc/source/whatsnew/v0.17.1.txt

@@ -122,3 +122,7 @@ Bug Fixes
 - Bug in ``to_excel`` with openpyxl 2.2+ and merging (:issue:`11408`)
 
 - Bug in ``DataFrame.to_dict()`` produces a ``np.datetime64`` object instead of ``Timestamp`` when only datetime is present in data (:issue:`11327`)
+
+
+
+- Bug in the link-time error caused by C ``inline`` functions on FreeBSD 10+ (with ``clang``) (:issue:`10510`)

setup.py

@@ -11,6 +11,7 @@
 import shutil
 import warnings
 import re
+import platform
 from distutils.version import LooseVersion
 
 # versioning
@@ -289,7 +290,10 @@ def run(self):
 
 
 class CheckingBuildExt(build_ext):
-    """Subclass build_ext to get clearer report if Cython is necessary."""
+    """
+    Subclass build_ext to get clearer report if Cython is necessary.
+    Also, add some platform based compiler flags.
+    """
 
     def check_cython_extensions(self, extensions):
         for ext in extensions:
@@ -302,8 +306,27 @@ def check_cython_extensions(self, extensions):
 
     def build_extensions(self):
         self.check_cython_extensions(self.extensions)
+        self.add_gnu_inline_flag(self.extensions)
         build_ext.build_extensions(self)
 
+    def add_gnu_inline_flag(self, extensions):
+        '''
+        Add CFLAGS `-fgnu89-inline` for clang on FreeBSD 10+
+        '''
+        if not platform.system() == 'FreeBSD':
+            return
+
+        try:
+            bsd_release = float(platform.release().split('-')[0])
+        except ValueError:  # unknow freebsd version
+            return
+
+        if bsd_release < 10:  # 9 or earlier still using gcc42
+            return
+
+        for ext in extensions:
+            ext.extra_compile_args += ['-fgnu89-inline']
+
 
 class CythonCommand(build_ext):
     """Custom distutils command subclassed from Cython.Distutils.build_ext