Set the minimum Perl version #9

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@paultcochrane

Setting the minimum Perl version in the dist's metadata is considered good
practice (see for instance
CPANTS and Neil
Bower's blog post concerning the
issue
).
This change thus also fixes the meta_yml_declares_perl_version core
kwalitee issue.

This PR is submitted in the hope that it is helpful. If it needs to be changed in any way, please just let me know and I'll update it and resubmit.

Set the minimum Perl version
Setting the minimum Perl version in the dist's metadata is considered good
practice (see for instance
[CPANTS](http://cpants.cpanauthors.org/dist/Crypt-OpenSSL-RSA) and [Neil
Bower's blog post concerning the
issue](http://blogs.perl.org/users/neilb/2014/08/specify-the-min-perl-version-for-your-distribution.html)).
This change thus also fixes the `meta_yml_declares_perl_version` core
kwalitee issue.
@toddr

This comment has been minimized.

Show comment
Hide comment
@toddr

toddr Apr 14, 2018

Merged via toddr@01baa09

ok to close.

toddr commented Apr 14, 2018

Merged via toddr@01baa09

ok to close.

akiym pushed a commit to akiym/Crypt-OpenSSL-RSA that referenced this pull request Apr 16, 2018

Merge pull request #9 from akiym/fix-configure-args
Crypt::OpenSSL::Guess is required at configure phase
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment