Skip to content

Test libre #4211

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 4 commits into from
Closed

Test libre #4211

wants to merge 4 commits into from

Conversation

reaperhulk
Copy link
Member

This tests #4210 with libre 2.7.2 in jenkins

Sp1l and others added 4 commits April 29, 2018 21:20
@Sp1l
Fix build with LibreSSL 2.7
5768b3b 
Tested with
  - OpenSSL 1.1.0h
  - LibreSSL 2.7.2
on 2.1.4 version

Signed-off-by: Bernard Spil <[email protected]>
@Sp1l
Fix pep8 issues
2c44d40 
Signed-off-by: Bernard Spil <[email protected]>
@Sp1l
Merge 2c44d40 into ca2055f
7b3a56c
@reaperhulk
add jenkins test
207c908
@Sp1l
Copy link
Contributor

Sp1l commented Apr 30, 2018

All packages are OK on FreeBSD. Cross-tested

  • Python 2.7 & 3.6
  • FreeBSD 10.4 & 11.1
  • amd64 & i386
  • OpenSSL 1.0.1u & 1.0.2o & 1.1.0h & 1.1.1-pre5 & LibreSSL 2.7.2

Build logs (poudriere) on https://keg.brnrd.eu/ (enumerating all 32 logs here was a bit much)

@reaperhulk
Copy link
Member Author

The CI is failing on LibreSSL 2.7.2 here. It looks like the defines probably aren't quite right. There are several symbols not present in 2.7.2 (You can see them in the error output from jenkins)

@dropwhile
Copy link

Does something need to be added here so libressl also takes the Cryptography_HAS_GENERIC_DTLS_METHOD = 0 path?

@reaperhulk reaperhulk mentioned this pull request May 10, 2018
Closed
@vishwin vishwin mentioned this pull request May 13, 2018
Closed
@Sp1l
Copy link
Contributor

Sp1l commented May 13, 2018

I've been digging and found some more issues where LibreSSL is missing methods. Expect an update to the 4210 issue shortly.

@Sp1l
Copy link
Contributor

Sp1l commented May 13, 2018

Just pushed an update to #4210 that fixes all 'implicit declaration' warnings during build and allows make test to run.

@reaperhulk
Copy link
Member Author

We do our libre tests in travis now so this PR is not necessary to test the 2.7.x PR.

@reaperhulk reaperhulk closed this May 22, 2018
@reaperhulk reaperhulk deleted the test-libre branch November 3, 2018 01:25
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@reaperhulk @Sp1l @dropwhile