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@6852b20 #3617

Merged
merged 29 commits into from Sep 29, 2022
Merged

Merge with pypa/distutils@6852b20 #3617

merged 29 commits into from Sep 29, 2022

Conversation

jaraco
Copy link
Member

@jaraco jaraco commented Sep 29, 2022

  • Use path.Path for changing the cwd temporarily.
  • Remove needless assert renderings. Any decent test runner will provide this detail.
  • Remove reference to duplicate issue.
  • Refactor _is_python_source_dir using pathlib.
  • Use 'samefile' to check for same file, because they may not be the same path. Fixes 3.4 release fails tests #180.
  • Split test into two tests.
  • Use pathlib
  • Remove special case for TestCore.write_setup.
  • Replace TESTFN with temp_file fixture.
  • Remove 'cleanup_testfn', unused.
  • Remove distracting comments
  • In test_sysconfig, prefer fixtures to TESTFN.
  • Remove cleanup_testfn, no longer used.
  • Replace TESTFN with fixture.
  • Remove empty logic branch.
  • Use path.Path for changing the cwd temporarily.
  • Remove reliance on TESTFN in test_dist
  • Remove TESTFN from py38compat, no longer needed.
  • xfail srcdir_simple on Windows
  • Remove reliance on os_helper in test_build_ext
  • Remove reliance on change_cwd
  • Add compatibility shim for subprocess on Python 3.7 on Windows
  • Extract _save_cwd for saving the current working directory.
  • Prefer a temp_cwd fixture to reduce indentation.
  • Remove reliance on create_empty_file
  • Add test for PermissionError. Ref [BUG] Change to pathlib causes Permission Errors when checking for config files distutils#181.
  • Use os.path.isfile for checking if path.is_file, suppresses exceptions. Fixes [BUG] Change to pathlib causes Permission Errors when checking for config files distutils#181.

Summary of changes

Closes

Pull Request Checklist

@jaraco jaraco merged commit 401c184 into main Sep 29, 2022
@jaraco jaraco deleted the distutils-6852b20 branch September 29, 2022 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Change to pathlib causes Permission Errors when checking for config files 3.4 release fails tests
1 participant