Fixup non-optional use of IO::Socket::IP by nhorman · Pull Request #27941 · openssl/openssl · GitHub | Latest TMZ Celebrity News & Gossip | Watch TMZ Live
Skip to content

Fixup non-optional use of IO::Socket::IP #27941

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Fixup non-optional use of IO::Socket::IP #27941

wants to merge 1 commit into from
+24 −3

Conversation

nhorman
Copy link
Contributor

@nhorman nhorman commented Jul 1, 2025

IO::Socket::IP is an optionally used package in our perl scripts, and a recent change of mine used it unilaterally, causing breakage on older perl installations. Fix it up to use it optionally again, falling back to IO::Socket::INET when needed.

Fixes #27940

@nhorman
Fixup non-optional use of IO::Socket::IP
a40a56f 
IO::Socket::IP is an optionally used package in our perl scripts, and a
recent change of mine used it unilaterally, causing breakage on older
perl installations.  Fix it up to use it optionally again, falling back
to IO::Socket::INET when needed.

Fixes openssl#27940
@nhorman nhorman requested review from levitte, t8m and mattcaswell July 1, 2025 18:22
@nhorman nhorman self-assigned this Jul 1, 2025
@nhorman nhorman added the branch: master Merge to master branch label Jul 1, 2025
@nhorman nhorman marked this pull request as draft July 1, 2025 18:22
@t8m t8m added branch: 3.2 Merge to openssl-3.2 branch: 3.3 Merge to openssl-3.3 branch: 3.4 Merge to openssl-3.4 branch: 3.5 Merge to openssl-3.5 approval: review pending This pull request needs review by a committer triaged: bug The issue/pr is/fixes a bug severity: regression The issue/pr is a regression from previous released version tests: exempted The PR is exempt from requirements for testing labels Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@levitte levitte Awaiting requested review from levitte

@t8m t8m Awaiting requested review from t8m

@mattcaswell mattcaswell Awaiting requested review from mattcaswell

At least 2 approving reviews are required to merge this pull request.

Assignees

@nhorman nhorman

Labels
approval: review pending This pull request needs review by a committer branch: master Merge to master branch branch: 3.2 Merge to openssl-3.2 branch: 3.3 Merge to openssl-3.3 branch: 3.4 Merge to openssl-3.4 branch: 3.5 Merge to openssl-3.5 severity: regression The issue/pr is a regression from previous released version tests: exempted The PR is exempt from requirements for testing triaged: bug The issue/pr is/fixes a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

New mandatory test dependency on perl module IO::Socket::IP
2 participants
@nhorman @t8m

TMZ Celebrity News – Breaking Stories, Videos & Gossip

Looking for the latest TMZ celebrity news? You've come to the right place. From shocking Hollywood scandals to exclusive videos, TMZ delivers it all in real time.

Whether it’s a red carpet slip-up, a viral paparazzi moment, or a legal drama involving your favorite stars, TMZ news is always first to break the story. Stay in the loop with daily updates, insider tips, and jaw-dropping photos.

🎥 Watch TMZ Live

TMZ Live brings you daily celebrity news and interviews straight from the TMZ newsroom. Don’t miss a beat—watch now and see what’s trending in Hollywood.