Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge with pypa/distutils@75ed79d #3366

merged 31 commits into from
Jun 12, 2022

Merge with pypa/distutils@75ed79d #3366

merged 31 commits into from
Jun 12, 2022


Copy link

@jaraco jaraco commented Jun 12, 2022

  • TST: Test that find_library_file works with Cygwin libs
  • Update name patterns for Cygwin libraries.
  • Fixed case of CC config variable not being defined.
  • Fix finding headers when cross compiling
  • Move more of the GCC detection to the is_gcc method.
  • Remove check for undefined CC var.
  • Inline the first option.
  • Update Python versions to include Python 3.11.
  • Remove tests for SO config var, no longer present in Python 3.11.
  • Mark test as xfail. Ref easy_install (via bdist_egg) adds to PEP 420 namespace packages #148.
  • 👹 Feed the hobgoblins (delint).
  • Add flake8 config from jaraco/skeleton for compatibility with black.
  • Skip string normalization when using black.
  • ⚫ Fade to black.
  • 👹 Feed the hobgoblins (delint).
  • Remove compatibility for Python 3.5 (except tests).
  • Remove compatibility for Python 3.5 in tests.
  • Avoid re-using 'prefix' variable.
  • Extract separate functions for get_python_inc{}, now that OS treatment diverges even more.
  • Extract functions for _get_python_inc for config or prefix variants. Use value algebra to simplify the logic.
  • ⚫ Fade to black.
  • ⚫ Fade to black.
  • Extract function for posix python build.

Summary of changes


Pull Request Checklist

DWesl and others added 30 commits May 4, 2022 12:16
It worked for setuptools<60 and doesn't with setuptools>=60; I would like it to keep working.
- `/usr/lib/lib${name}.a` Static library, needed at link time, embedded for run time
- `/usr/lib/lib${name}.dll.a` Import library, needed at link time, sets up run-time redirections to DLL
- `/usr/bin/cyg${name}.dll` Dynamically-linked library, needed at run time.  Extension used for python C extension modules.

Replaces #140
Test that find_library_file works with Cygwin libraries
When cross-compiling third-party extensions, get_python_inc() may be called to
return the path to Python's headers. However, it uses the sys.prefix or
sys.exec_prefix of the build Python, which returns paths pointing to build
system headers when instead we really want the host system headers.

To fix this, we use the INCLUDEPY and CONFINCLUDEPY conf variables, which can
be configured to point at host Python by setting _PYTHON_SYSCONFIGDATA_NAME.
The existing behavior is maintained on non-POSIX platforms or if a prefix is
manually specified.
Fixed Issue on Windows+MSYS2 with unsafe handling of CC config variable.
Add testing for Python 3.11
Fix finding headers when cross compiling
@jaraco jaraco merged commit eb75ea6 into main Jun 12, 2022
@jaraco jaraco deleted the feature/distutils-75ed79d branch June 12, 2022 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

4 participants