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

add/change to "use warnings;" #2

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@cakirke

cakirke commented Jan 13, 2016

No description provided.

@jkeenan

This comment has been minimized.

Show comment
Hide comment
@jkeenan

jkeenan Jan 16, 2016

Owner

Partly at the prodding which your earlier correspondence provided, I made some changes to EU::MM and released v0.55 to CPAN earlier this month. As a result, this patch did not apply cleanly to master when I tried to apply it via 'git am'. Could you re-draw the patch against master? Thank you very much.

Owner

jkeenan commented Jan 16, 2016

Partly at the prodding which your earlier correspondence provided, I made some changes to EU::MM and released v0.55 to CPAN earlier this month. As a result, this patch did not apply cleanly to master when I tried to apply it via 'git am'. Could you re-draw the patch against master? Thank you very much.

@jkeenan

This comment has been minimized.

Show comment
Hide comment
@jkeenan

jkeenan Jan 16, 2016

Owner

Follow-up: It only took me a few minutes to do a search-and-replace. So I made the change to 'use warnings;' throughout and just pushed it to github (though I'll wait before doing a new CPAN release). Thank you very much.

Owner

jkeenan commented Jan 16, 2016

Follow-up: It only took me a few minutes to do a search-and-replace. So I made the change to 'use warnings;' throughout and just pushed it to github (though I'll wait before doing a new CPAN release). Thank you very much.

@jkeenan jkeenan closed this Jan 16, 2016

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