Skip to content

Merge with distutils@129480b#3475

Merged
jaraco merged 32 commits into
mainfrom
distutils-129480b
Jul 31, 2022
Merged

Merge with distutils@129480b#3475
jaraco merged 32 commits into
mainfrom
distutils-129480b

Conversation

@jaraco

@jaraco jaraco commented Jul 31, 2022

Copy link
Copy Markdown
Member
  • Move import of _msvccompiler into the imports section. Skip only those tests that fail based on winreg.
  • Remove test suite construction in distutils.tests
  • Remove unittest cruft
  • Replace EnvironGuard with a pytest fixture.
  • DOC: Extend Windows rpath error message with likely cause
  • Implement ZLIB_SUPPORT as a pytest fixture.
  • 👹 Feed the hobgoblins (delint).
  • ⚫ Fade to black.
  • Extract expected values into a single list.
  • 👹 Feed the hobgoblins (delint).
  • 👹 Feed the hobgoblins (delint).
  • 👹 Feed the hobgoblins (delint).
  • 👹 Feed the hobgoblins (delint).
  • 👹 Feed the hobgoblins (delint).
  • 👹 Feed the hobgoblins (delint).
  • 👹 Feed the hobgoblins (delint).
  • 👹 Feed the hobgoblins (delint).
  • Replace if/else with simple algebra.
  • Consolidate 'format_command' and 'format_commands' now that order is retained in dicts.
  • Consolidate messaging when runtime_library_dirs is indicated.
  • Fix issue with Setuptools integration.
  • Rewrite comment in imperative voice. Use bulleted list for clarity. Short circuit when functionality isn't required, rather than leaving a dangling empty branch.
  • Extract _library_root to disentangle that concern from the other calculations.
  • Rewrite the comment to focus on the mechanics and not the author.
  • Eliminate repetition by iterating over the problem space.
  • Calculate the CFLAGS match early and simply include that in the choice on whether to change the root.
  • Assign result to a check and use that to consolidate return into a single expression.
  • ⚫ Fade to black.

Summary of changes

Closes

Pull Request Checklist

jaraco and others added 30 commits July 30, 2022 18:42
Extend the error message for attempting to call `runtime_library_dir_option` on Windows with the most likely way I have found to trigger the call, creating an `Extension` that specifies `runtime_library_dirs`.

Hopefully this will give users something to search for in their `setup.py` and a starting point for what to change.  Inspired by #3450.
DOC: Extend Windows rpath error message with likely cause
…hort circuit when functionality isn't required, rather than leaving a dangling empty branch.
jaraco added 2 commits July 31, 2022 14:53
…s a dictionary, but fall back to the dual dict/list model for compatibility with stdlib.
@jaraco jaraco merged commit ccb0666 into main Jul 31, 2022
@jaraco jaraco deleted the distutils-129480b branch July 31, 2022 23:33
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.

4 participants