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 distutils@d82d926 #3609

merged 23 commits into from Sep 24, 2022

Merge with distutils@d82d926 #3609

merged 23 commits into from Sep 24, 2022


Copy link

@jaraco jaraco commented Sep 24, 2022

  • Revert "Exclude Python 3.11 on macOS due to lack of wheels. Ref pyobjc failing to build on Python 3.11 distutils#165."
  • Restore metadata tests (not discovered due to class name).
  • Add DISTUTILS_EXTRA_CONFIG option for passing setup.cfg overrides during build
  • Fix name in test
  • Prefer monkeypatch for setting environment variable
  • Use tmp_path and to build files.
  • Use functools.lru_cache to memoize check_environ.
  • Use pathlib for generating paths and generate the paths in a separate function, consolidating 'is_file' check.
  • Update docs to reference environment variable.
  • Rename environment variable to DIST_EXTRA_CONFIG, decoupling it from the name of the implementation.
  • Simplify logic in test_custom_pydistutils
  • Inline variables used once
  • Remove meaningless dot from config files
  • Extract a temp_home fixture.
  • Re-use temp_home in test_find_config_files_disable
  • Extract property for pydistutils.cfg name
  • Move property to a module attribute and re-use jaraco.path for simpler tests.
  • Use jaraco.path for more tests
  • Fix warning in test

Summary of changes


Pull Request Checklist

jaraco and others added 23 commits September 24, 2022 09:06
Add DIST_EXTRA_CONFIG option for passing setup.cfg overrides during build
@jaraco jaraco merged commit e0dac17 into main Sep 24, 2022
32 checks passed
@jaraco jaraco deleted the distutils-d82d926 branch September 24, 2022 19:16
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

2 participants