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

openssl/3.2.1 dgram support for for QNX #24359

Open
klausholstjacobsen opened this issue May 10, 2024 · 2 comments
Open

openssl/3.2.1 dgram support for for QNX #24359

klausholstjacobsen opened this issue May 10, 2024 · 2 comments
Labels
help wanted triaged: question The issue contains a question

Comments

@klausholstjacobsen
Copy link

klausholstjacobsen commented May 10, 2024

I know that official support for QNX was removed in 2018.
I have however been building OpenSSL for qnx successfully until now.

In 3.2.x I am however seeing errors:

crypto/bio/bss_dgram.c: In function 'pack_local':
crypto/bio/bss_dgram.c:1178:13: error: 'struct in_pktinfo' has no member named 'ipi_spec_dst'
         info->ipi_spec_dst      = local->s_in.sin_addr;

Would it be ok to add a PR with __QNX__ among the ifdefs in crypto/bio/bs_dgram.c:1177 so that:

#   if !defined(OPENSSL_SYS_WINDOWS) && !defined(OPENSSL_SYS_CYGWIN)
        info->ipi_spec_dst      = local->s_in.sin_addr;
#   endif

...becomes...

#   if !defined(OPENSSL_SYS_WINDOWS) && !defined(OPENSSL_SYS_CYGWIN) && !defined(__QNX__)
        info->ipi_spec_dst      = local->s_in.sin_addr;
#   endif

or is any reference to QNX not allowed?

Regards
Klaus

@klausholstjacobsen klausholstjacobsen added the issue: question The issue was opened to ask a question label May 10, 2024
@tom-cosgrove-arm
Copy link
Contributor

tom-cosgrove-arm commented May 10, 2024

Looking at the PR (5756) that removed the support, the reason given was "cannot find contributors to get their approval for the license change", and @mattcaswell (still on the OTC and OMC) stated

In the event that someone later re-contributes QNX support I think we should accept it as a "bug" fix

so it seems extremely likely that such a small PR would be accepted. So please: raise that as a PR.

@mattcaswell
Copy link
Member

Our platform policy says this:

Support for a new platform should only be added if it is being adopted as a primary, secondary or community platform.

https://www.openssl.org/policies/general/platform-policy.html

A community platform is described like this:

Definition: Platforms that one or more members of the OpenSSL community have volunteered to support. May or may not be in project CI. Members of the community providing support do not have to be committers.

So in order re-add support for QNX it would have to become a community platform, which would need someone to volunteer to support it on an ongoing basis. That can be done by creating a PR against the general-policies repo, like this recent example:

openssl/general-policies#66

@mattcaswell mattcaswell added triaged: question The issue contains a question and removed issue: question The issue was opened to ask a question labels May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted triaged: question The issue contains a question
Projects
None yet
Development

No branches or pull requests

4 participants