Skip to content

Conversation

@aykevl
Copy link
Member

@aykevl aykevl commented Sep 25, 2020

This allows the following packages to pass tests:

  • crypto/des
  • crypto/rc4
  • encoding/hex

... which are two outdated ciphers and one potentially useful package. Still, it's a good thing to get as many packages to pass tests, if nothing else it helps to catch regressions in the compiler.

@aykevl aykevl force-pushed the test-benchmark-stubs branch from e189bb5 to 7883712 Compare September 25, 2020 20:21
@aykevl
Copy link
Member Author

aykevl commented Sep 25, 2020

Removed crypto/rc4 as it doesn't work on Go 1.11.

This allows the following packages to pass tests:

  * crypto/des
  * encoding/hex

I have not included crypto/rc4 as it doesn't pass tests on Go 1.11 (but
it works on later versions).
@aykevl aykevl force-pushed the test-benchmark-stubs branch from 7883712 to 8d82e59 Compare October 4, 2020 16:23
@aykevl
Copy link
Member Author

aykevl commented Oct 4, 2020

Rebased, after a conflict with #1411

@deadprogram
Copy link
Member

Looking good, now merging. Thanks @aykevl

@deadprogram deadprogram merged commit d8dbe57 into dev Oct 9, 2020
@deadprogram deadprogram deleted the test-benchmark-stubs branch October 9, 2020 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants