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

Fix dependency upgrades #15740

Merged
merged 6 commits into from
Jan 10, 2025
Merged

Conversation

AlanCoding
Copy link
Member

@AlanCoding AlanCoding commented Jan 10, 2025

SUMMARY

This includes

just want to put this up to document

For people visiting from the future, pypa/setuptools#4457 is the setuptools PR that causes the pin added here. It modifies sys.path and this causes import errors, because the irc lib has dependencies overlapping with setuptools _vendor directory, and they conflict, functionally.

Next pallets/markupsafe#478 put a minimum pin on setuptools, but this is non-conflicting with our other needs because there is a version range that satisfies markupsafe's new requirement while still pre-dating the sys.path hack.

I don't have a particularly good reference for the cryptography situation, other than general release notes, which make a ton of notes about specific OpenSSL versions.

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • API

@github-actions github-actions bot added the dependencies Pull requests that update a dependency file label Jan 10, 2025
@AlanCoding AlanCoding changed the title Fix dep update test alan Fix dependency upgrades Jan 10, 2025
@AlanCoding AlanCoding marked this pull request as ready for review January 10, 2025 20:58
@AlanCoding AlanCoding enabled auto-merge (squash) January 10, 2025 21:11
@AlanCoding AlanCoding merged commit c1572af into ansible:devel Jan 10, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants