-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add 32-bit Windows to our primary platform list #61
Conversation
We already have 64-bit Windows on our primary platform list, and 32-bit and 64-bit Linux. We should probably also have 32-bit Windows on the list.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might want to add it in our CIs too
We must. The question is how easy it is for us. Do we have 32-bit architecture and Windows available via GitHub actions? @quarckster, wdyt? It looks like Microsoft still supports 32-bit architecture:
Windows 11 doesn't run on 32-bits, as it doesn't satisfy the minimum RAM requirement. |
We already have 32 bit Windows builds in the CI, so IMO no action needed on the CI side. Of course we do not build & test them on 32 bit Windows installations but that should not matter much as hopefully the 32 bit support on 64 bit Windows installs is fully compatible. On the other hand we do not really have proper 32 bit Linux testing. |
Note that we currently have a baseline Windows support level of XP ( Actually, XP is not even listed on our platform list except for ia64 as a community platform (which is just comical...). So it seems like we have an incongruity here. Either we should add XP to our platform list and test it in CI or we should allow bumping minimum I'd be interested in data from our community on how many people are relying on OpenSSL working on XP at this point. |
I am opening a vote on this one:
|
Voting +1
|
Windows XP has been EOL for ages, I'm for bumping the minimum version requirement.
32-bit openssl builds 64-bit Windows is a real use case scenario and as noticed @t8m we have it in the CI. An action item for this issue should be updating https://www.openssl.org/policies/general-supplemental/platforms.html and bumping |
Note that this is a PR not an issue - the PR updates the platforms list. |
@arapov - can I take your approval of the PR as a +1 on the vote? |
This comment was marked as outdated.
This comment was marked as outdated.
Yes, my approval of this PR is +1 |
Vote has been closed and passed:
I have recorded the vote and pushed this PR. |
We already have 64-bit Windows on our primary platform list, and 32-bit and 64-bit Linux. We should probably also have 32-bit Windows on the list.