Skip to content
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.

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

Remove unnecessary Junit dependency #38

Closed
wants to merge 1 commit into from

Conversation

robertszuba
Copy link

The prov module declares the Junit library as a runtime dependency. The library is only needed for the test step during compile time - the testCompile dependency is already added in the top level build.gradle of the project. The runtime dependency definition can generate warnings in the end-app builds.

@rtyley
Copy link
Owner

rtyley commented Jan 30, 2018

Thanks for this contribution @robertszuba - as it happens Spongy Castle is a repackage of Bouncy Castle, targeted at Android, with no substantial differences to the code (just renaming of packages, etc). So if you've got an issue that relates to the underlying Bouncy Castle codebase, it's better to raise the issue with the GitHub repository for Bouncy Castle at https://github.com/bcgit/bc-java .

It turns out that this issue was fixed in Bouncy Castle with bcgit@9f7320cd0d4e, which made it into r1rv59 - so when I make the corresponding Spongy Castle release the issue will be fixed.

@rtyley rtyley closed this Jan 30, 2018
@jbuhacoff
Copy link

You can build your own BC/SC jars from the current version (1.62), see my comment on issue #34.

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