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

fix const strip (build error) - openssl.cpp #134

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Manu-sh
Copy link

@Manu-sh Manu-sh commented Sep 19, 2023

fix this conversion error:

/home/user/greenpass_qrcode_reader/build/_deps/cose-c-src/src/openssl.cpp:2007:76: error: invalid conversion from ‘const ec_key_st*’ to ‘EC_KEY*’ {aka ‘ec_key_st*’} [-fpermissive]
 2007 |                                 EC_KEY *peckeyPublic = EVP_PKEY_get0_EC_KEY(evpPublic);
      |                                                        ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~
      |                                                                            |
      |                                                                            const ec_key_st*

Schermata del 2023-09-19 12-41-00

fix this conversion error:

```make
/home/user/greenpass_qrcode_reader/build/_deps/cose-c-src/src/openssl.cpp:2007:76: error: invalid conversion from ‘const ec_key_st*’ to ‘EC_KEY*’ {aka ‘ec_key_st*’} [-fpermissive]
 2007 |                                 EC_KEY *peckeyPublic = EVP_PKEY_get0_EC_KEY(evpPublic);
      |                                                        ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~
      |                                                                            |
      |                                                                            const ec_key_st*
```
@Manu-sh Manu-sh changed the title Update openssl.cpp fix const strip - openssl.cpp Sep 19, 2023
@Manu-sh Manu-sh changed the title fix const strip - openssl.cpp fix const strip (build error) - openssl.cpp Sep 19, 2023
@Manu-sh
Copy link
Author

Manu-sh commented Sep 19, 2023

hmm that smart CI.. just add manually if you want that still build on arch linux, I don't spend time to figure out what's wrong with CI

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

Successfully merging this pull request may close these issues.

1 participant