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

Added warnings and updated license in makefile #3

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@royalkiran007

royalkiran007 commented Jan 23, 2015

The link http://cpants.cpanauthors.org/author/ABH had two flags. Tried to resolve them by adding warnings and editing LICENSE. Work done as a part of CPAN pull request challenge.

royalkiran007 added some commits Jan 23, 2015

Update CA.pm
added warnings pragma to check whether it throws warnings
Update locate-file.t
added warnings to test script
Update Makefile.PL
Edited LICENSE. removed gpl_1 and gpl_2
@gisle

This comment has been minimized.

Show comment
Hide comment
@gisle

gisle Jan 23, 2015

I rechecked the license. The last PEM file seems to be under the MPL 2.0 license. That was updated in commit d5e9139. Not sure what the value of the warnings for 4 lines of code are.

gisle commented on b41bc57 Jan 23, 2015

I rechecked the license. The last PEM file seems to be under the MPL 2.0 license. That was updated in commit d5e9139. Not sure what the value of the warnings for 4 lines of code are.

@abh abh closed this Jan 4, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment