Skip to content

genserv: error out if the openssl tool is missing#16928

Closed
bagder wants to merge 4 commits intomasterfrom
bagder/gencert-no-openssl
Closed

genserv: error out if the openssl tool is missing#16928
bagder wants to merge 4 commits intomasterfrom
bagder/gencert-no-openssl

Conversation

@bagder
Copy link
Member

@bagder bagder commented Apr 2, 2025

Ref: #16926

@bagder bagder added the tests label Apr 2, 2025
@github-actions github-actions bot added the script label Apr 2, 2025
@bagder bagder mentioned this pull request Apr 2, 2025
Closed
@vszakats
Copy link
Member

vszakats commented Apr 2, 2025

Ah, my attempt: #16929

It honors the actual openssl used in the commands and also detects
some cases of bad (old, fork) openssl cases.

vszakats added a commit to vszakats/curl that referenced this pull request Apr 2, 2025
@vszakats
update error message via @bagder curl#16928
34f2372
@bagder
Copy link
Member Author

bagder commented Apr 2, 2025

Closed in favor of #16929

@bagder bagder closed this Apr 2, 2025
@bagder bagder deleted the bagder/gencert-no-openssl branch April 2, 2025 22:26
vszakats added a commit that referenced this pull request Apr 3, 2025
@vszakats
genserv.pl: fail with a message if openssl is missing or failing
3d4e4a1 
Reported-by: Tomas Volf
Fixes #16926
Follow-up to 44341e7 #16824
Ref: #16928
Co-authored-by: Daniel Stenberg
Closes #16929
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

script tests

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bagder @vszakats