Remove Crypto++ support #3283

Merged
merged 1 commit into from Feb 4, 2016

Projects

None yet

4 participants

@pieterlexis
Member

No description provided.

@zeha
Collaborator
zeha commented Jan 22, 2016

Could you rebase this on top of master, so we get a working travis build (while xs is down). (Changes LGTM)

@ahupowerdns
Member

is there a reason to remove this beyond "it is not better than the other dependencies"?

@pieterlexis
Member

we now require OpenSSL (see #3281), so cryptopp primitives are never used. (discussed with @Habbie, @zeha and @mind04 on IRC)

@Habbie Habbie merged commit ae24274 into PowerDNS:master Feb 4, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@pieterlexis pieterlexis deleted the pieterlexis:bye-bye-cryptopp branch Sep 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment