ENH: improve resiliency in distutils.build_ext.finalize_options

Author: neutrinoceros

newsfragments/5083.bugfix.rst

@@ -0,0 +1,2 @@
+Fixed a bug where calling ``build_ext.finalize_options`` more than once would
+raise an exception.

setuptools/_distutils/command/build_ext.py

@@ -266,17 +266,17 @@ def finalize_options(self) -> None:  # noqa: C901
         # specified by the 'define' option will be set to '1'.  Multiple
         # symbols can be separated with commas.
 
-        if self.define:
+        if isinstance(self.define, str):
             defines = self.define.split(',')
             self.define = [(symbol, '1') for symbol in defines]
 
         # The option for macros to undefine is also a string from the
         # option parsing, but has to be a list.  Multiple symbols can also
         # be separated with commas here.
-        if self.undef:
+        if isinstance(self.undef, str):
             self.undef = self.undef.split(',')
 
-        if self.swig_opts is None:
+        if not self.swig_opts:
             self.swig_opts = []
         else:
             self.swig_opts = self.swig_opts.split(' ')

setuptools/tests/test_build_ext.py

@@ -178,6 +178,17 @@ def C(file):
         assert example_stub.startswith(f"{build_lib}/mypkg/__pycache__/ext1")
         assert example_stub.endswith(".pyc")
 
+    def test_finalize_options_multiple_call(self):
+        """
+        Regression test. Check that calling build_ext.finalize_options
+        more than once doesn't raise an exception.
+        """
+        extension = Extension('spam.eggs', ['eggs.c'], py_limited_api=True)
+        dist = Distribution(dict(ext_modules=[extension]))
+        cmd = build_ext(dist)
+        cmd.finalize_options()
+        cmd.finalize_options()
+
 
 class TestBuildExtInplace:
     def get_build_ext_cmd(self, optional: bool, **opts) -> build_ext: