]> granicus.if.org Git - python/commitdiff
bpo-30912: Don't check the content of ffi.h (GH-2687)
authorShlomi Fish <shlomif@shlomifish.org>
Wed, 6 Sep 2017 20:19:19 +0000 (23:19 +0300)
committerZachary Ware <zachary.ware@gmail.com>
Wed, 6 Sep 2017 20:19:19 +0000 (13:19 -0700)
Various platforms have various methods of handling multiarch libffi which probably won't match the previously looked-for defines.  Now we just make sure that ffi.h is available.

setup.py

index 3c7c9049ed68ec60ba9de508529102a27907fe40..79f96989a40d4c474d8832b846eadaad9d888edd 100644 (file)
--- a/setup.py
+++ b/setup.py
@@ -2021,16 +2021,9 @@ class PyBuildExt(build_ext):
             ffi_inc = find_file('ffi.h', [], inc_dirs)
         if ffi_inc is not None:
             ffi_h = ffi_inc[0] + '/ffi.h'
-            with open(ffi_h) as f:
-                for line in f:
-                    line = line.strip()
-                    if line.startswith(('#define LIBFFI_H',
-                                        '#define ffi_wrapper_h')):
-                        break
-                else:
-                    ffi_inc = None
-                    print('Header file {} does not define LIBFFI_H or '
-                          'ffi_wrapper_h'.format(ffi_h))
+            if not os.path.exists(ffi_h):
+                ffi_inc = None
+                print('Header file {} does not exist'.format(ffi_h))
         ffi_lib = None
         if ffi_inc is not None:
             for lib_name in ('ffi', 'ffi_pic'):