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

Crypt::PRNG does not provide a VERSION #1

Merged
merged 1 commit into from
Feb 12, 2015
Merged

Crypt::PRNG does not provide a VERSION #1

merged 1 commit into from
Feb 12, 2015

Conversation

reneeb
Copy link
Contributor

@reneeb reneeb commented Feb 12, 2015

Unfortunately Crypt::PRNG does not provide a VERSION. This is why installation of Mojolicious::Plugin::OAuth2::Server fails:

==> Found dependencies: Crypt::PRNG, Test::Exception
Searching Crypt::PRNG on cpanmetadb ...
Found Crypt::PRNG  which doesn't satisfy 0.021.
...
-> FAIL Installing the dependencies failed: Module 'Crypt::PRNG' is not installed
-> FAIL Bailing out the installation for Mojolicious-Plugin-OAuth2-Server-0.07.

Unfortunately Crypt::PRNG does not provide a VERSION. This is why installation of Mojolicious::Plugin::OAuth2::Server fails:

```
==> Found dependencies: Crypt::PRNG, Test::Exception
Searching Crypt::PRNG on cpanmetadb ...
Found Crypt::PRNG  which doesn't satisfy 0.021.
...
-> FAIL Installing the dependencies failed: Module 'Crypt::PRNG' is not installed
-> FAIL Bailing out the installation for Mojolicious-Plugin-OAuth2-Server-0.07.
```
@leejo
Copy link
Collaborator

leejo commented Feb 12, 2015

Ah nuts. I think changing to CryptX is overkill, given how much it installs. I will probably fudge this to not require a specific version of Crypt::PRNG (will need to check that all version provide the necessary function). I'll probably raise an issue against Crypt::PRNG too to get the VERSION information fixed.

@leejo
Copy link
Collaborator

leejo commented Feb 12, 2015

Actually having looked at how much Crypt::PRNG installs i think switching to CryptX isn't a problem. Will merge, thanks!

leejo pushed a commit that referenced this pull request Feb 12, 2015
Crypt::PRNG does not provide a VERSION, use CryptX instead
@leejo leejo merged commit 61f7dec into payprop:master Feb 12, 2015
leejo added a commit that referenced this pull request Feb 12, 2015
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 5ace8fb on reneeb:patch-1 into * on leejo:master*.

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.

None yet

3 participants