Add manylinux_2_24 support (#595)

test/test_before_build.py

@@ -7,7 +7,7 @@ from . import test_projects, utils
 
 project_with_before_build_asserts = test_projects.new_c_project(
     setup_py_add=textwrap.dedent(r'''
-        import sys, os
+        import os
 
         # assert that the Python version as written to pythonversion.txt in the CIBW_BEFORE_BUILD step
         # is the same one as is currently running.

test/test_docker_images.py

@@ -7,7 +7,7 @@ from . import test_projects, utils
 
 dockcross_only_project = test_projects.new_c_project(
     setup_py_add=textwrap.dedent(r'''
-        import os, sys
+        import os
 
         # check that we're running in the correct docker image as specified in the
         # environment options CIBW_MANYLINUX1_*_IMAGE

test/test_manylinuxXXXX_only.py

@@ -9,6 +9,9 @@ from . import test_projects, utils
 project_with_manylinux_symbols = test_projects.new_c_project(
     spam_c_top_level_add=textwrap.dedent(r'''
         #include <malloc.h>
+        #include <stdlib.h>
+        #include <stdint.h>
+        #include <math.h>
 
         #if !defined(__GLIBC_PREREQ)
         #error "Must run on a glibc linux environment"
@@ -19,19 +22,21 @@ project_with_manylinux_symbols = test_projects.new_c_project(
         #endif
     '''),
     spam_c_function_add=textwrap.dedent(r'''
-        #if defined(__GLIBC_PREREQ) && __GLIBC_PREREQ(2, 17)  /* manylinux2014 is glibc 2.17 */
-            // secure_getenv is only available in manylinux2014, ensuring
-            // that only a manylinux2014 wheel is produced
-            secure_getenv("NON_EXISTING_ENV_VARIABLE");
+        #if defined(__GLIBC_PREREQ) && __GLIBC_PREREQ(2, 24)
+            // nextupf is only available in manylinux_2_24+
+            sts = (int)nextupf(0.0F);
+        #elif defined(__GLIBC_PREREQ) && __GLIBC_PREREQ(2, 17)  /* manylinux2014 is glibc 2.17 */
+            // secure_getenv is only available in manylinux2014+
+            sts = (int)(intptr_t)secure_getenv("NON_EXISTING_ENV_VARIABLE");
         #elif defined(__GLIBC_PREREQ) && __GLIBC_PREREQ(2, 10)  /* manylinux2010 is glibc 2.12 */
             // malloc_info is only available on manylinux2010+
-            malloc_info(0, stdout);
+            sts = malloc_info(0, stdout);
         #endif
     '''),
 )
 
 
-@pytest.mark.parametrize('manylinux_image', ['manylinux1', 'manylinux2010', 'manylinux2014'])
+@pytest.mark.parametrize('manylinux_image', ['manylinux1', 'manylinux2010', 'manylinux2014', 'manylinux_2_24'])
 def test(manylinux_image, tmp_path):
     if utils.platform != 'linux':
         pytest.skip('the docker test is only relevant to the linux build')
@@ -46,7 +51,7 @@ def test(manylinux_image, tmp_path):
     # CFLAGS environment variable is necessary to fail on 'malloc_info' (on manylinux1) during compilation/linking,
     # rather than when dynamically loading the Python
     add_env = {
-        'CIBW_ENVIRONMENT': 'CFLAGS="$CFLAGS -Werror=implicit-function-declaration"',
+        'CIBW_ENVIRONMENT': 'CFLAGS="$CFLAGS -O0 -Werror=implicit-function-declaration"',
         'CIBW_MANYLINUX_X86_64_IMAGE': manylinux_image,
         'CIBW_MANYLINUX_I686_IMAGE': manylinux_image,
         'CIBW_MANYLINUX_PYPY_X86_64_IMAGE': manylinux_image,
@@ -57,14 +62,14 @@ def test(manylinux_image, tmp_path):
     if manylinux_image == 'manylinux1':
         # We don't have a manylinux1 image for PyPy
         add_env['CIBW_SKIP'] = 'pp*'
-    elif manylinux_image == 'manylinux2014':
-        # We don't have a manylinux2014 image for PyPy (yet?)
-        add_env['CIBW_SKIP'] = 'cp27* pp*'  # Python 2.7 not available on manylinux2014
+    elif manylinux_image in {'manylinux2014', 'manylinux_2_24'}:
+        # We don't have a manylinux2014 / 'manylinux_2_24' image for PyPy (yet?)
+        add_env['CIBW_SKIP'] = 'cp27* pp*'  # Python 2.7 not available on manylinux2014 / 'manylinux_2_24'
     actual_wheels = utils.cibuildwheel_run(project_dir, add_env=add_env)
 
     expected_wheels = [w for w in utils.expected_wheels('spam', '0.1.0', manylinux_versions=[manylinux_image])]
-    if manylinux_image == 'manylinux2014':
+    if manylinux_image in {'manylinux2014', 'manylinux_2_24'}:
         expected_wheels = [w for w in expected_wheels if '-cp27' not in w]
-    if manylinux_image in ['manylinux1', 'manylinux2014']:
+    if manylinux_image in {'manylinux1', 'manylinux2014', 'manylinux_2_24'}:
         expected_wheels = [w for w in expected_wheels if '-pp' not in w]
     assert set(actual_wheels) == set(expected_wheels)

test/test_pep518.py

@@ -7,7 +7,6 @@ basic_project = test_projects.new_c_project(
     setup_py_add=textwrap.dedent(
         """
         # Will fail if PEP 518 does work
-        import sys
         import requests
         if sys.version_info < (3, 6, 0):
             assert requests.__version__ == "2.22.0", "Requests found but wrong version ({0})".format(requests.__version__)

test/test_projects/c.py

@@ -60,12 +60,21 @@ MOD_INIT(spam)
 '''
 
 SETUP_PY_TEMPLATE = r'''
+import sys
 from setuptools import setup, Extension
 
 {{ setup_py_add }}
 
+libraries = []
+if sys.platform.startswith('linux'):
+    libraries.extend(['m', 'c'])
+
 setup(
-    ext_modules=[Extension('spam', sources=['spam.c'])],
+    ext_modules=[Extension(
+        'spam',
+        sources=['spam.c'],
+        libraries=libraries,
+    )],
     {{ setup_py_setup_args_add | indent(4) }}
 )
 '''

test/test_ssl.py

@@ -5,7 +5,6 @@ from . import test_projects, utils
 project_with_ssl_tests = test_projects.new_c_project(
     setup_py_add=textwrap.dedent(r'''
         import ssl
-        import sys
 
         if sys.version_info[0] == 2:
             from urllib2 import urlopen