Skip to content

Test libre#4211

Closed
reaperhulk wants to merge 4 commits intopyca:masterfrom
reaperhulk:test-libre
Closed

Test libre#4211
reaperhulk wants to merge 4 commits intopyca:masterfrom
reaperhulk:test-libre

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
Tested with
  - OpenSSL 1.1.0h
  - LibreSSL 2.7.2
on 2.1.4 version

Signed-off-by: Bernard Spil <brnrd@FreeBSD.org>
Signed-off-by: Bernard Spil <brnrd@FreeBSD.org>
@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?

@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.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants