chore: replace Portable BC with BC Crypto #614
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
Description of changes:
It does not appear that BouncyCastle is going to continue development of
Portable.BouncyCastle
, which apparently is a labor of love/brilliance by one developer.Instead, they seem to be committing to
BouncyCastle.Cryptography
.See: https://www.bouncycastle.org/csharp/#DOWNLOAD20200
Squash/merge commit message, if applicable:
chore: replace Portable BC with BC Crypto
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Due to dafny-lang/dafny#2500, Traits are dangerous:
{:termination false}
?The override checks on
the specifications on
a class' functions/methods/etc. validating
that specifications are
at least as strong as those on
the traits it implements
are not working correctly when
that trait is defined in a different module
(and hence must have
{:termination false}
on it).As such, if either (1.) or (2.) is true: