Skip to content

Clearly document the OpenSSL version required #320

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

Open
blipper opened this issue Apr 17, 2021 · 2 comments
Open

Clearly document the OpenSSL version required #320

blipper opened this issue Apr 17, 2021 · 2 comments
Assignees
Labels
check for relevance Needs check whether this issue is still relevant and if so what to do about it

Comments

@blipper
Copy link
Contributor

blipper commented Apr 17, 2021

Is your feature request related to a problem? Please describe.
There are requirements for files/tests that are not present in earlier versions of OpenSSL.

Describe the solution you'd like
Document that exact minimum version needed.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@MMory
Copy link
Member

MMory commented Apr 26, 2021

Maybe we need to provide some internal dummy header? This only refers to the test cases of the type state analysis for openssl, for which we don't need any actual functionality of openssl.

@pdschubert pdschubert assigned pdschubert and meret-boe and unassigned pdschubert Jan 6, 2022
@pdschubert
Copy link
Member

Hi @meret-boe, please implement Martin's suggestion. For the test code in test/llvm_test_code/openssl/* various OpenSSL declarations are required to get it to compile to LLVM IR. We are currently including the respective header files from the user's system-wide OpenSSL installation. Let's create a dummy header instead that contains the necessary declarations such that we can remove the dependencies on OpenSSL.

Please create a dummy file for each original header file (cf. README.txt files).

@MMory MMory added the check for relevance Needs check whether this issue is still relevant and if so what to do about it label Dec 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
check for relevance Needs check whether this issue is still relevant and if so what to do about it
Projects
None yet
Development

No branches or pull requests

4 participants