Skip to content

build: fix OpenSSL version detection - #59353

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
richardlau:opensslv
Aug 6, 2025
Merged

build: fix OpenSSL version detection#59353
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
richardlau:opensslv

Conversation

@richardlau

Copy link
Copy Markdown
Member

Fix OpenSSL version detection in configure.py when pkg-config is used to configure building with a shared OpenSSL library.

Fixes: #59249 (comment)


cc @jasnell

Fix OpenSSL version detection in `configure.py` when `pkg-config` is
used to configure building with a shared OpenSSL library.
@nodejs-github-botnodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels Aug 4, 2025

@jasnelljasnell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@richardlaurichardlau added the request-ci Add this label to start a Jenkins CI on a PR. label Aug 4, 2025
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 4, 2025
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@richardlaurichardlau added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Aug 4, 2025
@aduh95aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 5, 2025
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 6, 2025
@nodejs-github-bot
nodejs-github-bot merged commit 2b4a09e into nodejs:mainAug 6, 2025
70 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 2b4a09e

@targostargos added dont-land-on-v20.x dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. dont-land-on-v24.x PRs that should not land on the v24.x-staging branch and should not be released in v24.x. labels Aug 8, 2025
mete0rfish pushed a commit to mete0rfish/node-contribute that referenced this pull request Aug 9, 2025
Fix OpenSSL version detection in `configure.py` when `pkg-config` is
used to configure building with a shared OpenSSL library.
PR-URL: nodejs#59353
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@richardlau
richardlau deleted the opensslv branch August 21, 2025 14:43
@RafaelGSSRafaelGSS mentioned this pull request Oct 9, 2025
9 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.buildIssues and PRs related to build files or the CI.dont-land-on-v22.xPRs that should not land on the v22.x-staging branch and should not be released in v22.x.dont-land-on-v24.xPRs that should not land on the v24.x-staging branch and should not be released in v24.x.needs-ciPRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@richardlau@nodejs-github-bot@panva@jasnell@lpinca@targos@aduh95