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

Fixed invalid version check mechanism #70

Closed
wants to merge 3 commits into from
Closed

Fixed invalid version check mechanism #70

wants to merge 3 commits into from

Conversation

htalaco
Copy link

@htalaco htalaco commented Oct 7, 2021

The old mechanism would fail to check for version 3.0.0, this would fix that issue.

@htalaco
Copy link
Author

htalaco commented Oct 11, 2021

As I figured out, PR not taken into account here!

@htalaco htalaco closed this Oct 11, 2021
@michaelpq
Copy link
Contributor

As I figured out, PR not taken into account here!

I think that your patch does not improve the code quality anyway. I would have added an extra check on the three digits instead.

@michaelpq
Copy link
Contributor

michaelpq commented Oct 19, 2021

I am checking the contents of the OpenSSL installers to see if any changes are required for 3.0.0, and I'll send a patch to merge to upstream. Thanks for the report!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants